Add class of service chooser for Google Flights Agent

if left blank Google defaults to coach. do you think it explicitly needs to
be defined if no pref?
On Sat, Nov 5, 2016 at 10:15 AM Dominik Sander<EMAIL_ADDRESS>wrote:

@dsander commented on this pull request.
In app/models/agents/google_flights_agent.rb
https://github.com/cantino/huginn/pull/1778#pullrequestreview-7311800:

@@ -69,6 +70,7 @@ def default_options
form_configurable :origin, type: :string
form_configurable :destination, type: :string
form_configurable :date, type: :string

form_configurable :preferredCabin, type: :array, values: %w(COACH PREMIUM_COACH BUSINESS FIRST)

Don't we also need a "no preference" value if the user does not care about
the cabin?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cantino/huginn/pull/1778#pullrequestreview-7311800,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AASBwVeX6vM4-2AvYiq1kEuefTBxLAMYks5q7Fc0gaJpZM4KpvCN
.

Will existing agents send coach or nil until the user edits them?

@bobbysteel The problem I see is with the way form_configurable dropdowns works one can not set it back to an empty value after selecting a preference once.

@dsander not sure I follow you. the empty value corresponds in their API to coach, so if someone wants to set back to default, it will revert to coach.
this agent has a ton of other configurable fields like preferred airline, etc., and I'd like to flesh it out so I can say look for BUSINESS on UA on a certain relative date period (say 120 days out). most of them have defaults in the API itself, so not sure how to best represent this other than how I did it w/ this one trial.
@cantino i've no idea - i'm really rusty in Ruby and doing my best to understand. regardless of the behavior, it should have the same results from the API as undefined defaults to coach, so either way it won't have impact.

@dsander at least in Chrome for me, with the form_configurable widget, there's a drop down option to manual, which lets you reset the row to nil again if chosen.

@dsander at least in Chrome for me, with the form_configurable widget, there's a drop down option to manual, which lets you reset the row to nil again if chosen.

True, that works. I am thinking about adding a include_blank option when defining a form configurable array. Then it would be available for all agents.

That's useful and would be nice in general
On Sun, Nov 6, 2016 at 7:44 PM Dominik Sander<EMAIL_ADDRESS>wrote:

@dsander https://github.com/dsander at least in Chrome for me, with the
form_configurable widget, there's a drop down option to manual, which lets
you reset the row to nil again if chosen.
True, that works. I am thinking about adding a include_blank option when
defining a form configurable array. Then it would be available for all
agents.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cantino/huginn/pull/1778#issuecomment-258704927, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AASBwZ8l0AzROo8P_qIfuV63T5R1PvCJks5q7i4SgaJpZM4KpvCN
.

Any other comments here? I'd like to extend this for all the options in this API like airline choice etc
Seems like no choice is ok in other settings as api does not require those optional fields

Seems like it'd be okay to add the other options as long as sending nothing, or sending "" to the API, always defaults to your default value.

@cantino are you comfortable merging this in with a view towards me adding the full options later on? These are the fields most needed now, and I will do the rest at a later point...

ok i took that check out @cantino

Thanks @bobbysteel!


Implement todo lists balancing mechanism

This PR adds a mechanism for the creation of balanced todo lists.
Background
This mechanism is needed for fixing instability problems in some fuzzers which led to timeouts.
The root cause of the timeout is that starting with a certain iteration the fuzzer (segmatch) was unable to find a clear solution for the PIPs from the todo list.
In a particular case (#784) the timeout was caused by the fact that the  PIPs with the CLK_HROW_CK_IN_ source wire, e.g. CLK_HROW_BOT_R_CK_BUFG_CASCO7.CLK_HROW_CK_IN_L8 had 3 candidate bits while the SEGMATCH_ARGS had the limiation set to 2 ( -c 2). This is because in fact only two bits are for the CLK_HROW_BOT_R_CK_BUFG_CASCO7.CLK_HROW_CK_IN_L8 PIP and the third bit is for CLK_HROW_CK_IN_L8_ACTIVE.
In order for segmatch to solve the bits correctly we need another PIP with exactly the same clock source wire on the todo list to disambiguate the _ACTIVE from the PIP bits.
Implementation
The mechanism was added to the int_maketodo.py script responsible for the creation of todo lists.
It was designed to be generic in order to create more balanced todo list based on few parameters specified with the use of additional switches (--balance-wire-re , --balance-wire-direction and --balance-wire-cnt)
Example todo list:

CLK_HROW_TOP_R.CLK_HROW_TOP_R_CK_BUFG_CASCO0.CLK_HROW_CK_IN_R8
CLK_HROW_TOP_R.CLK_HROW_TOP_R_CK_BUFG_CASCO21.CLK_HROW_CK_IN_R9

Command to create a balanced todo file:

python3 ../int_maketodo.py --sides "bot_r,top_r" --pip-type clk_hrow --seg-type clk_hrow --re ".*" --balance-wire-re "^CLK_HROW_CK_IN_" --balance-wire-direction "src" --balance-wire-cnt "2"

Final, balanced todo list:

CLK_HROW_TOP_R.CLK_HROW_TOP_R_CK_BUFG_CASCO27.CLK_HROW_CK_IN_R8
CLK_HROW_TOP_R.CLK_HROW_TOP_R_CK_BUFG_CASCO21.CLK_HROW_CK_IN_R9
CLK_HROW_TOP_R.CLK_HROW_TOP_R_CK_BUFG_CASCO0.CLK_HROW_CK_IN_R8
CLK_HROW_TOP_R.CLK_HROW_TOP_R_CK_BUFG_CASCO7.CLK_HROW_CK_IN_R9

Another example:
Initial todo:

CLK_HROW_TOP_R.CLK_HROW_TOP_R_CK_BUFG_CASCO0.CLK_HROW_CK_IN_R8
CLK_HROW_TOP_R.CLK_HROW_TOP_R_CK_BUFG_CASCO0.CLK_HROW_CK_IN_R9

Command:

python3 ../int_maketodo.py --sides "bot_r,top_r" --pip-type clk_hrow --seg-type clk_hrow --re ".*" --balance-wire-re "^CLK_HROW_TOP_R_CK_BUFG_CASCO" --balance-wire-direction "dst" --balance-wire-cnt "4"

Balanced todo:

CLK_HROW_TOP_R.CLK_HROW_TOP_R_CK_BUFG_CASCO0.CLK_HROW_CK_IN_R3
CLK_HROW_TOP_R.CLK_HROW_TOP_R_CK_BUFG_CASCO0.CLK_HROW_CK_IN_R9
CLK_HROW_TOP_R.CLK_HROW_TOP_R_CK_BUFG_CASCO0.CLK_HROW_CK_IN_R8
CLK_HROW_TOP_R.CLK_HROW_TOP_R_CK_BUFG_CASCO0.CLK_HROW_CK_IN_L0

CI is green, i.e. it doesn't break anything.

@mithro Just added some comment. Could you take a look if it's clear enough?


get OSS build to pass, add VS Studio Code support

passed tests

Hi Alan and Luca, I updated the PR based on the review. I removed those
comments altogether as the latest OS installed libgoogle-glog-dev seems to
do the trick even without any CMakeLists.txt reference. To answer Luca's
question, the find_package was failing because there were no .cmake files
being found.. which appears common for those things not built under the FB
deps manifests. The latest gihub workflows still pass so it looks good to
go from here. lmk if you see anything else. G
On Tue, Oct 22, 2024 at 9:15 PM afrind @.***> wrote:

@.**** commented on this pull request.
In CMakeLists.txt
https://github.com/facebookexperimental/moxygen/pull/4#discussion_r1811620926
:

+# We can NOT use Glog latest version, is NOT compatible with moxygen - XXX is this comment still relevant
+# find_package(glog REQUIRED) - XXX this fails OSS build

Can you wrap this to 80 chars? This is our convention.
In CMakeLists.txt
https://github.com/facebookexperimental/moxygen/pull/4#discussion_r1811621118
:

+# We can NOT use Glog latest version, is NOT compatible with moxygen - XXX is this comment still relevant
+# find_package(glog REQUIRED) - XXX this fails OSS build

@lnicco https://github.com/lnicco asks:
I guess I don't fully understand why for the author find_package would
fail but pkg_check_modules passes.
—
Reply to this email directly, view it on GitHub
https://github.com/facebookexperimental/moxygen/pull/4#pullrequestreview-2386671275,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB7T3IAQIJ3HJ3K5WCLHMPLZ432C3AVCNFSM6AAAAABQHVQ636VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGOBWGY3TCMRXGU
.
You are receiving this because you authored the thread.Message ID:
@.***>


Connect to a published port from the outside world : difference between PWD and PWK

Hi,
There is a difference in the FAQ between "master" tag and "k8s" tag concerning access to published port :
master README.md (PWD)
If you need to access your services from outside, use the following URL pattern http://ip<hyphen-ip>-<session_jd>-<port>.direct.labs.play-with-docker.com (i.e: http://ip-2-135-3-b8ir6vbg5vr00095iil0-8080.direct.labs.play-with-docker.com).

k8s README.md (PWK)
If you need to access your services from outside, use the following URL pattern http://pwd<underscore_ip>-<port>.<host#>.labs.play-with-docker.com (i.e: http://pwd10_2_135_3-80.host3.labs.play-with-docker.com/).

hyphen or underscore ?
'ip' or 'pwd' prefix ?
'play-with-docker.com' or 'play-with-k8s.com' suffix ?

I am trying to display kubernetes dashboard on PWK as advised in #258 but does not manage to have the right URL...
Are these differences normal and what is the right configuration to access K8S dashboard after deploying it and starting 'kubectl proxy --port=8080' ?
Thanks

Try deploying the dashboard service and expose it as either NodePort or LoadBalancer. You can use this command to dynamically change it in the kubectl apply.
curl -L -s https://raw.githubusercontent.com/kubernetes/dashboard/master/src/deploy/alternative/kubernetes-dashboard.yaml  | sed 's/targetPort: 9090/targetPort: 9090\n  type: LoadBalancer/' |    kubectl apply -f -
That should populate the blue badge at the top so you can click in there

No blue badge is not getting displayed. Can you please help

Try deploying the dashboard service and expose it as either NodePort or LoadBalancer. You can use this command to dynamically change it in the kubectl apply.
curl -L -s https://raw.githubusercontent.com/kubernetes/dashboard/master/src/deploy/alternative/kubernetes-dashboard.yaml | sed 's/targetPort: 9090/targetPort: 9090\n type: LoadBalancer/' | kubectl apply -f -
That should populate the blue badge at the top so you can click in there

@marcosnils This command displays the following error message :
[node1 install]$ curl -L -s https://raw.githubusercontent.com/kubernetes/dashboard/master/src/deploy/alternative/kubernet
es-dashboard.yaml | sed 's/targetPort: 9090/targetPort: 9090\n type: LoadBalancer/' | kubectl apply -f -
serviceaccount "kubernetes-dashboard" created
role "kubernetes-dashboard-minimal" created
rolebinding "kubernetes-dashboard-minimal" created
deployment "kubernetes-dashboard" created
error: error converting YAML to JSON: yaml: line 13: did not find expected key

Do you have a new way to be able to use the dashboard in PWK ?
Plus, I ma sorry but my initial question about the differences between PWD and PWK instructions in README remains unanswered... Can you please explain these differences ?

Quick suggestions:

The URL https://raw.githubusercontent.com/kubernetes/dashboard/master/src/deploy/alternative/kubernet es-dashboard.yaml  is no longer valid. The correct link is:

kubectl apply -f https://raw.githubusercontent.com/kubernetes/dashboard/master/aio/deploy/recommended/kubernetes-dashboard.yaml
Also, the default port is 8443 and NOT 9090. Hence you might need to correct it to:
sed 's/targetPort: 8443/targetPort: 9090\n type: LoadBalancer/'

Hope it helps

This doesn't work as suggested
I see the external IP in pending state and no port shows up on the dashboard
I ran the command on PWK:
curl -L -s https://raw.githubusercontent.com/kubernetes/dashboard/master/aio/deploy/recommended/kubernetes-dashboard.yaml | sed 's/targetPort: 8443/targetPort: 9090\n  type: LoadBalancer/' | kubectl apply -f -

and checked on the service as follows:
node1 ~]$ kubectl -n kube-system get service kubernetes-dashboard
NAME                   TYPE           CLUSTER-IP       EXTERNAL-IP   PORT(S)         AGE
kubernetes-dashboard   LoadBalancer   <IP_ADDRESS>   <pending>     443:31593/TCP   8m

We are still not able to view blue badge. Please help

@rahgadda this documentation is a bit old. We'll try to fix it :)

@marcosnils - I also did not get the Blue Badge to access the K8 Dashboard from external IP, no external IP is assigned to me. also wen i run "kubectl proxy --port=8443" it is running for long time without exiting. Appreciate your help and guidance

how do they connect from abroad??


Bug: Meta Data Not Rehydrated when using persist and rehydrate state

Description
I encountered an issue while attempting to save and rehydrate the state using Persisted and Rehydrated State example. It appears that the meta data associated with the state is not rehydrated correctly after the page reloads or when the app is revisited, causing unexpected behavior in the application.
Expected result
When saving to local storage and then rehydrating the state, the meta data should be fully rehydrated along with the rest of the state. This means that the loaded meta data should accurately reflect the state's meta data at the time of saving, and the application should behave as expected without any loss of meta data after rehydration.
Actual result
Upon saving and rehydrating the state using the described process, I noticed that the meta data associated with the state is not correctly rehydrated. Instead, remains an empty object {}, leading to unexpected behavior in my application that relies on accurate meta data.
Reproduction
https://codesandbox.io/s/xstate-meta-bug-zdq29t
Additional context
I hope this information helps in identifying and resolving the issue. Please let me know if you require any additional information or clarification. Thank you for your attention to this matter!

This is fixed in XState v5 beta: https://codesandbox.io/s/xstate-meta-bug-forked-f2s2l3?file=/src/index.js
If you want, you can try your hand at fixing it in v4. It likely has to do with how the State is JSON-stringified (we're probably forgetting the meta value).

Great, thank you for providing the solution! I will bump up the packages to the beta version of XState v5 and test it out. If that resolves the issue, it's excellent news.
As for fixing the issue in XState v4, I will take a look at it as well, but please keep in mind that my time is limited because I take care of :baby: :baby:. So, I will investigate it further only if I have the opportunity and availability.
Once again, thank you for your help and guidance on this matter! I'll keep you updated on my progress if I can find the time to investigate further.

@bogminic I went ahead and quickly fixed this: #4169


Roll calculator improvement: cumulative stats

ms.space provides a way to see cumulative stats for modules like damage mods (i,e, dps output boost, which combines damage boost and rof boost). Roll calculator uses only basic stats so far.
However, for practical purposes I mostly care about cumulative stats. E.g. i want to see how often i will get +22.5%+ dps and 32.5- cpu for radiation sink.
I don't know how it can be calculated math-wise, but pretty sure there should be some algebraic way to solve that. Will write any details in comments if i come up with the solution.

example for 2-factor cumulative stats:

2D plane with x as damage modifier and y as rof modifier
area restricted by x-axis range [min dmg mod, max dmg mod] and y-axis range [min rof mod, max rof mod] is area of possibilities, defines 100% chance to roll. Min and max here are absolute min and max for rolls, not user-defined limits.
area between user-set min dmgboost and user-set dmg boost functions (see [1]) is area of chance. Obviously it should be within area of possible values (point 2), User might want to set specific dmgmod / rofmod values which further restricts area of chance
total chance for dmgmod + rofmod is area of chance / area of possibilities

So algebraic solution is as easy as finding out area below max and min graphs. Where graph is straight line you just take rectangular value, where graph is dmgboost function you just take definite integral of it over range where it applies. You subtract max area vs min area - this is your chance area. Then you divide it by area of possibilities which yields you final combined chance, which takes into consideration all 3 user-set ranges (cumulative dmg boost, dmg modifier, rof modifier).
As a side-note, ability to control dmg modifier/rof modifier separately even if cumulative dmg boost is controlled makes sense due to stacking penalties: you get better results if you have 2 22% damage modules where 1 has high rof roll and 1 has high dmg roll, compared to 2 damage modules which have equal rolls on both.
Oh and also i don't know if there're 3-factor cumulative stats, which might make it a bit more complex.
[1]
dmgboost = (dmgmod / rofmod - 1) * 100
dmgmod / rofmod = dmgboost / 100 + 1
rofmod = dmgmod / (dmgboost / 100 + 1)


Consistent box-sizing configuration

🧹 Tech Debt
We are currently a bit scattershot in where and how we configure the box-sizing mode of nimble-components.
We should be more consistent like with some helper that can do a minimal CSS reset in shadow roots for box-sizing.
We likely should also have recommendations for apps to configure their box-sizing modes more globally but that is outside the scope of this work item.

Assigned to @rajsite to add more detail to the proposal:

what mechanism do we use to do the css reset
what's the default value we want for box-sizing
what's the per-component work


added customization

Pull request for this issue https://github.com/ProgramEquity/amplify/issues/370

@paramsiddharth

That's true 😅..@paramsiddharth . Let me change that

added the commit

@DietBepis1 Does the PR look good to merge, Glenn? :)

@DietBepis1 & @paramsiddharth   done 🎅🎅

Awesome! Great job!

Thanks @DietBepis1  and @paramsiddharth

Integration test bugs, refer to these logs.
@DietBepis1 Does this make sense to you?

All the failing tests seem to be a consequence of this JWT failure.

Expand
FAIL server/__tests__/integration/lob.test.js
  ● Test suite failed to run

    TypeError: jwt is not a function

      3 | const { domain, audience } = require('./config/env.dev')
      4 |
    > 5 | const checkJwt = jwt({
        |                  ^
      6 |   secret: jwksRsa.expressJwtSecret({
      7 |     cache: true,
      8 |     rateLimit: true,

      at Object.jwt (server/auth/check-jwt.js:5:18)
      at Object.require (server/routes/api/authentication.js:9:[22](https://github.com/ProgramEquity/amplify/actions/runs/3955575083/jobs/6773981158#step:5:23))

FAIL server/__tests__/integration/representatives.test.js
  ● Test suite failed to run

    TypeError: jwt is not a function

      3 | const { domain, audience } = require('./config/env.dev')
      4 |
    > 5 | const checkJwt = jwt({
        |                  ^
      6 |   secret: jwksRsa.expressJwtSecret({
      7 |     cache: true,
      8 |     rateLimit: true,

      at Object.jwt (server/auth/check-jwt.js:5:18)
      at Object.require (server/routes/api/authentication.js:9:22)

PASS server/__tests__/integration/axios-cache-interceptor.test.js (11.8[34](https://github.com/ProgramEquity/amplify/actions/runs/3955575083/jobs/6773981158#step:5:35) s)
FAIL server/__tests__/integration/checkout.test.js
  ● Test suite failed to run

    TypeError: jwt is not a function

      3 | const { domain, audience } = require('./config/env.dev')
      4 |
    > 5 | const checkJwt = jwt({
        |                  ^
      6 |   secret: jwksRsa.expressJwtSecret({
      7 |     cache: true,
      8 |     rateLimit: true,

      at Object.jwt (server/auth/check-jwt.js:5:18)
      at Object.require (server/routes/api/authentication.js:9:22)

Test Suites: 3 failed, 1 passed, 4 total
Tests:       3 passed, 3 total
Snapshots:   0 total
Time:        14.[63](https://github.com/ProgramEquity/amplify/actions/runs/3955575083/jobs/6773981158#step:5:64)4 s
Ran all test suites matching /server\/__tests__\/integration\//i.
Error: Process completed with exit code 1.

@DietBepis1 Hey Glenn, need some urgent help here deciding what needs to be done to merge this PR. It has taken way too long and despite resolving the conflicts and merging main in multiple times I'm unable to get the integration tests to pass. We've merged PRs with failing tests before but right now the checks succeed for new PRs and I'm afraid merging this PR without proper consideration could lead to them breaking too.

Oops! Somehow the integration tests are still failing. @DietBepis1 Really need your help here!


[FRG] - Allow data formatting

I'm mostly looking for ideas on this problem. It feels like data formatting would be a reasonable thing for a reporting engine to manage, but I want to make sure it's done in an extendable way, so that multiple formats can be used or configured for a specific type.
When editing bands:

I think it would make sense to see a "Format" option when you're editing a print or a cell. The value would get run through whatever format function is specified before printing. This would need to apply to totals as well. The Fluent engine has .totalFormatter(), but I think this type of functionality goes beyond that.
For the extendibility part, I'm curious what your thoughts are. I think that formatters would need to be able to be defined for when the report is generated. For example, in our system, we allow the user to choose between 24 hour time, and 12 hour time. So when our displayTime() method is called, it decides how it's supposed to format based on the user's preferences.
A simple, probably dumb way to do it, would be to manually define a list of functions in the configuration of the editor, then those could be chosen from in a dropdown added to the screenshot above. Then those functions would be called and used when the report is generated. If possible, those functions would need to support Promises (async/await).
That's just one idea though, like I said, I'm very open to better ones.

So then basically the report editor can show a select with formatDate and SomeOtherFormatter, then the values are sent to the formatter; and the formatter then calls the callback with the formatted value.   In side the function you can use promises/await/callback, or whatever method you want to do async functions....
Maybe this could be passed in as part of the Data or ReportData.

I think that's similar, some pseudo-code just so I understand.
Client/Editor
const frg = new FluentReportsGenerator({
  id: 'fluentReportsEditor',
  data: [],
  report: {},
  formatterNames: ['formatDate', 'SomeOtherFormatter']
});

Server/Renderer
const reportBuilder = require('fluentreports/lib/fluentReportsBuilder').ReportBuilder;
const report = new reportBuilder({}, []);
report.formatters = {
  'formatDate': (inValue, callback) => {
    callback(`Time: ${inValue}`);
  },
  'SomeOtherFormatter': (inValue, callback) => {
    callback(`Other: ${inValue}`);
  }
};
report.render();

I think that's essentially what you're saying, right?

Something else that came to my mind, that feels pretty dumb, but might work as well, is a simple data contract.
[
{amount: 123, amount-formatted: "$ 123.00"},
{amount: 4567, amount-formatted: "$ 4,567.00"}
]

Basically when any value is about to be printed, it checks for a -formatted version of the field, and uses that if it exists. There's some downsides to that. Efficiency, it puts the work on the developer to format all of their data, etc.
But one upside, is that I've been experimenting with using worker_threads to render reports. Getting 2+ seconds of blocking IO off the main thread sure is nice. To do it the first way, we'd have to go the route of a report worker pool, which is probably where we'll end up anyway.

I'm not stuck on either one of these, just throwing out ideas at this point.

As for your first part; basically yes -- however I think we would need to pass in the formatter function (even if it is a dummy function) to the generator as you can preview reports.  So it has to be able to run it.
As for passing in -formatted if you are wanting to do this; then all you have to do is pass it into the report generator and set the label/band to print that field; and the engine actually doesn't have to do anything...  :)
But it would be fairly easy for the generator engine to check for an -formatted value and pass it in instead.   The engine already does something similar to this for dealing with all printed items.

however I think we would need to pass in the formatter function (even if it is a dummy function) to the generator as you can preview reports. So it has to be able to run it.

At least in our case, I've overridden the preview functionality to be identical to running the report, so that would work the same either way.
My personal opinion is that's the direction we need to go. I like the simplicity from the report editor's perspective. As a user, I just:

Add a print/band
Choose a formatter function to use

And that's it. It complicates the server implementation slightly, but I think I'm happy to absorb the complication for the end user.
I want to do what's best for FluentReports as well though, so you may have an opinion on it as well.


Create odbskmb.txt

We will use PhpStorm for HTML, JavaScript, PHP and MySQL basics.

@lsvejdaodbskmb Pull request merged. Thank you!


Automate Readme.md

If some new contributor visits twindle, they will need to know the exact stack we're working on. This will help them decide if they want to join us or not. This is just a suggestion from me. We can market our project better through this. Refer sample image 1
We can also include one column of all our daily calls as a playlist and show like the second image.
Readme.md -> Edit this READme.md including the tech stack like below one.
Here is the sample image 1:

Here is the sample image 2:

Refer this Readme.md
Follow this tutorial if you have any doubt:

I am fine with this. We can merge the PR if any one raises it..

I can be responsible for this PR.

@RafaelBatman55 go ahead

Curious if this is working as expected?

this is also broken

I tried to keep in this list just the Google Meet's video playlist, but didn't work. 👎
Hopefully @Akshay2996 checked this problem and prepared PR #743 (already merged).

Yes I think so


modify some comment error,  manually reopen pr

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: cg82616424:white_check_mark: MRXLT:x: chengang06chengang06 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Already merged the private branches with some improvements

... maybe you like to have this is also in the origin?
Regards
Stefan

Apologies for taking as long to respond to your request. Thanks for sending the changes. Unfortunately, due to differences in formatting (mainly in XML files like pom.xml and servlet-context.xml) it's very difficult to verify what's actually changed. If there is any way you can merge the latest, resolve conflicts locally and re-submit, I will respond immediately. I also noticed that your code for FormController.java is a bit out of date. Try backing out any changes you have in that class first as it has changed significantly.


Gulp 4 Watch is Not Working as expected

Following code used to work with old syntax of defining tasks with gulp.task, but is not working anymore with the regular functions and the export.
gulpfile.js
function symlinkYaml...

function convertYamlToJson(done) {
    var doc = yaml.safeLoad(fs.readFileSync('./my.yaml'));
    fs.writeFileSync('./public/my.json', JSON.stringify(doc, null, "  "));
}

function watchYaml(done) {
    return gulp.watch("my.yaml", convertYamlToJson);
}

exports.default = gulp.series(symlinkYaml, watchYaml);

Command used to run script
gulp
What was expected:
Any change to my.yaml file should invoke convertYamlToJson.
What is happening:
convertYamlToJson is only called the first time my.yaml file changes.

https://gulpjs.com/docs/en/getting-started/async-completion#no-synchronous-tasks


server: subscriptions for validators and blocknumber not working

Failed to load resource: net::ERR_CONNECTION_TIMED_OUT
possibly related #271
I think in order to make the app always work we should just ping a full node for the current validator set as a fallback

it's back up,
however it does need to query then poll, as subscription listens for new writes (which may not happen till the next session).


Add mint-client-cli 'list-gateways' and 'switch-gateways' command

Introducing mint-client-cli commands for interacting with the gateways.
This PR proposes:

[x] mint-client-cli list-gateways to lists all gateways registered with the mint,
[x] mint-client-cli switch-gateways to activate a different gateway.

These commands allow interactions and management of gateways from CLI as proposed in Issue #366
Demo:

list-gateways shows all registered gateways, with their activation status.

$ mint-client-cli list-gateways

Found 1 registered gateways :
[
  {
    "mint_pub_key": "b1f4d07fe3d5a754d47dd58b9db2710ab292d125a9cba3a854c1524c92201135",
    "node_pub_key": "03c218a54e57553fa0a466900b5b7de4c51db421138084a52e45fbdfd21103e560",
    "api": "http://<IP_ADDRESS>:8080",
    "active": false
  }
]

switch-gateway <node_pub_key> shows all registered gateways, with their activation status. It might be useful to allow selection with either pubkey, so mint_pub_key would work as well

$ mint-client-cli switch-gateways 03c218a54e57553fa0a466900b5b7de4c51db421138084a52e45fbdfd21103e560

Successfully switched to gateway with the following details
{
  "mint_pub_key": "b1f4d07fe3d5a754d47dd58b9db2710ab292d125a9cba3a854c1524c92201135",
  "node_pub_key": "03c218a54e57553fa0a466900b5b7de4c51db421138084a52e45fbdfd21103e560",
  "api": "http://<IP_ADDRESS>:8080",
  "active": true
}

list-gateways shows all registered gateways, with their updated activation status after switch-gateways

$ mint-client-cli list-gateways

Found 1 registered gateways :
[
  {
    "mint_pub_key": "b1f4d07fe3d5a754d47dd58b9db2710ab292d125a9cba3a854c1524c92201135",
    "node_pub_key": "03c218a54e57553fa0a466900b5b7de4c51db421138084a52e45fbdfd21103e560",
    "api": "http://<IP_ADDRESS>:8080",
    "active": true
  }
]

Codecov Report

Merging #407 (f7f6bd0) into master (f91fb2f) will decrease coverage by 0.06%.
The diff coverage is 38.88%.

@@            Coverage Diff             @@
##           master     #407      +/-   ##
==========================================
- Coverage   75.33%   75.26%   -0.07%
==========================================
  Files          83       83
  Lines       10395    10408      +13
==========================================
+ Hits         7831     7834       +3
- Misses       2564     2574      +10

Impacted Files
Coverage Δ

client/cli/src/main.rs
0.76% <0.00%> (-0.06%)
:arrow_down:

client/client-lib/src/lib.rs
88.94% <87.50%> (+0.07%)
:arrow_up:

minimint/src/net/connect.rs
94.94% <0.00%> (-0.26%)
:arrow_down:

minimint/src/net/peers.rs
93.15% <0.00%> (ø)

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

Also, a commit like "fix lint errors" should just be squashed together with the commit that introduced the error (it's a minor fixup). A later commit should not undo changes from a previous one typically.

here some proposed changes: https://gist.github.com/NicolaLS/12e67484f1bbb90d98ed5e83aa0ad36f
notice removing the option will need refactor in more places than what this gist covers

or if we added a method activate(&self) on the gw, we could just use iter instead of iter_mut

very nice, I commented some optimizations, especially the Option<PubKey> dosn't really make sense to me, even though it looks like a less verbose way to just select the first one.

Thanks for the excellent review, @NicolaLS . I adopted some of the suggested the optimizations, especially on the iterators when selecting and  mutating the gateways.
Option<PubKey> as applied is a great save on network calls to get registered gateways. So I think we can keep it, but with full  expectations to iterate the code to more idiomatic signature for the gateways methods on client-lib as we develop more robust gateway management logic.

The first few commits aren't really atomic and just fix things from their predecessors. Can you squash them please? Also needs a rebase.

Generally when you need to change a previous commit you can use git rebase --interactive <start point> where <start point> is a reference to the earliest commit you might want to change (a hash or e.g. HEAD~5 for "the previous 5 commits").

Generally when you need to change a previous commit you can use git rebase --interactive <start point> where <start point> is a reference to the earliest commit you might want to change (a hash or e.g. HEAD~5 for "the previous 5 commits").

rebase went ok, now working through resolving conflicts and trimming the out-of-scope changes added!

LGTM. Nice first PR @okjodom!

Lot's of learning too!


REST

Ported the REST api.
I've adapted query generation to reuse the existing one from this project.
The idea is to squash all my commits and merge only two commits (one from Doug and one from me, in order to keep attribution to Doug as author).
Remaining tasks:

[ ] Error handling
[ ] Test coverage

cc @dougwettlaufer

I'm going to continue to review. How can I help with the REST endpoint? I can start writing tests?

I've fixed the issues from the initial review and rebase it against master, squashing all my commits to reduce the noise.

How can I help with the REST endpoint? I can start writing tests?

I'm starting to write some tests for insert/update/delete, so feel free to take ddl tests. I've created some basic structure for integration tests on rest_integration_test.go, we can direct push to this branch and do a final review afterwards, wdyt?

I think its ready to be reviewed. I've added support and test coverage for all scalar values.
There are some open questions and issues that should be addressed in different patches: https://github.com/datastax/cassandra-data-apis/issues?q=is%3Aissue+is%3Aopen+label%3Arest


[$250] Web - Sign Up - Concierge chat is not auto scrolled to the bottom after onboarding process

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.57-0
Reproducible in staging?: Y
Reproducible in production?: Y
Issue reported by: Applause Internal Team
Action Performed:

Open the staging.new.expensify.com website.
Sign up with a new account.
Select "Manage my team´s expenses"
Complete the onboarding process selecting any option on the following steps.
Verify you land on Concierge chat and the conversation is auto scrolled to the bottom.

Expected Result:
When landing on Concierge chat, the conversation should be auto scrolled to the bottom.
Actual Result:
When selecting "Manage my team´s expenses" during onboarding process, Concierge chat is not auto scrolled to the bottom when the user is redirected to it.
Workaround:
Unknown
Platforms:

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/8500eba1-cd7f-444e-b81d-fe44649f484f

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021853799179762467415
Upwork Job ID:<PHONE_NUMBER>762467415
Last Price Increase: 2024-11-05

Issue OwnerCurrent Issue Owner: @brunovjk

Proposal
Please re-state the problem that we are trying to solve in this issue.
Web - Sign Up - Concierge chat is not auto scrolled to the bottom after onboarding process
What is the root cause of that problem?
The root cause of the problem is we haven't provided lastVisibleActionCreated optimistically for the target chat Report here
https://github.com/Expensify/App/blob/210db71f7e549abf7c6b925e36029549ece83256/src/libs/actions/Report.ts#L3643-L3651
What changes do you think we should make in order to solve the problem?

We need to set the lastVisibleActionCreated to last action added to the targetChatReportID we optimistically add different report actions to the targetChatReport ID via different conditions and loop in the function and ideally it should be set to the `lastAction.created so
so we can create a variable
    let optimLastVisibleActionCreated = textCommentAction.created;

This change only can solve this issue but theoretically it should be set to the created of the last report action so we will reset the value in other places where other report actions are added to the report below this code like
https://github.com/Expensify/App/blob/210db71f7e549abf7c6b925e36029549ece83256/src/libs/actions/Report.ts#L3769
        optimLastVisibleActionCreated = videoCommentAction.created;

Take https://github.com/Expensify/App/blob/210db71f7e549abf7c6b925e36029549ece83256/src/libs/actions/Report.ts#L3643-L3651
to here
https://github.com/Expensify/App/blob/210db71f7e549abf7c6b925e36029549ece83256/src/libs/actions/Report.ts#L3796-L3797
by adding the lastvisibleactioncreated
 optimisticData.push({
        onyxMethod: Onyx.METHOD.MERGE,
        key: `${ONYXKEYS.COLLECTION.REPORT}${targetChatReportID}`,
        value: {
            lastMentionedTime: DateUtils.getDBTime(),
            hasOutstandingChildTask,
            lastVisibleActionCreated: optimLastVisibleActionCreated,
        },
    });

What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
Web - Sign Up - Concierge chat is not auto scrolled to the bottom after onboarding process
What is the root cause of that problem?
The root cause of the problem is we haven't provided lastVisibleActionCreated optimistically for the target chat Report here
https://github.com/Expensify/App/blob/210db71f7e549abf7c6b925e36029549ece83256/src/libs/actions/Report.ts#L3643-L3651
What changes do you think we should make in order to solve the problem?

We need to set the lastVisibleActionCreated to last action added to the targetChatReportID optimistically
so we can create a variable
    let optimLastVisibleActionCreated = textCommentAction.created;

This change only can solve this issue but theoretically it should be set to the created of the last report action so as video actions can be added we will reset the value inside the conditon here too (to ensure we set it to the last action)
https://github.com/Expensify/App/blob/210db71f7e549abf7c6b925e36029549ece83256/src/libs/actions/Report.ts#L3769
        optimLastVisibleActionCreated = videoCommentAction.created;

Take https://github.com/Expensify/App/blob/210db71f7e549abf7c6b925e36029549ece83256/src/libs/actions/Report.ts#L3643-L3651
to under here
https://github.com/Expensify/App/blob/210db71f7e549abf7c6b925e36029549ece83256/src/libs/actions/Report.ts#L3796-L3797
by adding the lastvisibleactioncreated
 optimisticData.push({
        onyxMethod: Onyx.METHOD.MERGE,
        key: `${ONYXKEYS.COLLECTION.REPORT}${targetChatReportID}`,
        value: {
            lastMentionedTime: DateUtils.getDBTime(),
            hasOutstandingChildTask,
            lastVisibleActionCreated: optimLastVisibleActionCreated,
        },
    });

What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
Web - Sign Up - Concierge chat is not auto scrolled to the bottom after onboarding process
What is the root cause of that problem?
The root cause of the problem is we haven't provided lastVisibleActionCreated optimistically for the target chat Report here
https://github.com/Expensify/App/blob/210db71f7e549abf7c6b925e36029549ece83256/src/libs/actions/Report.ts#L3643-L3651
What changes do you think we should make in order to solve the problem?

We need to set the lastVisibleActionCreated to last action added to the targetChatReportID optimistically
so we will set it to the last task report action and if it is not available we will set it to the video comment action if it is available or else the text comment action
 const lastVisibleActionCreated = tasksData.at(-1)?.taskReportAction.reportAction.created ?? videoCommentAction?.created ?? textCommentAction.created;

    const optimisticData: OnyxUpdate[] = [...tasksForOptimisticData];
    optimisticData.push(
        {
            onyxMethod: Onyx.METHOD.MERGE,
            key: `${ONYXKEYS.COLLECTION.REPORT}${targetChatReportID}`,
            value: {
                lastMentionedTime: DateUtils.getDBTime(),
                hasOutstandingChildTask,
                lastVisibleActionCreated,
            },
        },

What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
Web - Sign Up - Concierge chat is not auto scrolled to the bottom after onboarding process
What is the root cause of that problem?
The root cause of the problem is we haven't provided lastVisibleActionCreated optimistically for the target chat Report here
https://github.com/Expensify/App/blob/210db71f7e549abf7c6b925e36029549ece83256/src/libs/actions/Report.ts#L3643-L3651
What changes do you think we should make in order to solve the problem?

We need to set the lastVisibleActionCreated to last action added to the targetChatReportID optimistically
so we will set it to the last task report action timestamp and if it is not available we will set it to the video comment action otherwise to the text comment action's created timestamp.
 const lastVisibleActionCreated = tasksData.at(-1)?.taskReportAction.reportAction.created ?? videoCommentAction?.created ?? textCommentAction.created;

    const optimisticData: OnyxUpdate[] = [...tasksForOptimisticData];
    optimisticData.push(
        {
            onyxMethod: Onyx.METHOD.MERGE,
            key: `${ONYXKEYS.COLLECTION.REPORT}${targetChatReportID}`,
            value: {
                lastMentionedTime: DateUtils.getDBTime(),
                hasOutstandingChildTask,
                lastVisibleActionCreated,
            },
        },

What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
Web - Sign Up - Concierge chat is not auto scrolled to the bottom after onboarding process
What is the root cause of that problem?
The root cause of the problem is we haven't provided lastVisibleActionCreated optimistically for the target chat Report here
https://github.com/Expensify/App/blob/210db71f7e549abf7c6b925e36029549ece83256/src/libs/actions/Report.ts#L3643-L3651
What changes do you think we should make in order to solve the problem?

We need to set the lastVisibleActionCreated to last action added to the targetChatReportID optimistically
so we will set it to the last task report action timestamp and if it is not available we will set it to the video comment action otherwise to the text comment action's created timestamp.
    const lastVisibleActionCreated =
        tasksData.at(-1)?.completedTaskReportAction?.created ?? tasksData.at(-1)?.taskReportAction.reportAction.created ?? videoCommentAction?.created ?? textCommentAction.created;

    const optimisticData: OnyxUpdate[] = [...tasksForOptimisticData];
    optimisticData.push(
        {
            onyxMethod: Onyx.METHOD.MERGE,
            key: `${ONYXKEYS.COLLECTION.REPORT}${targetChatReportID}`,
            value: {
                lastMentionedTime: DateUtils.getDBTime(),
                hasOutstandingChildTask,
                lastVisibleActionCreated,
            },
        },

What alternative solutions did you explore? (Optional)

Possibly linked: https://github.com/Expensify/App/issues/49599

Possibly linked: #49599

Nope, that is completely different issue. 👍

https://github.com/user-attachments/assets/2eeca164-f078-4f88-bff5-c41fe1b1cb26

I’ll review the proposals shortly

@FitseTLT's proposal seems good to me, I tested the changes and it solves the problem:
https://github.com/user-attachments/assets/9e0dbb97-2710-4559-9044-70276f40c40a
🎀👀🎀 C+ reviewed

Proposal LGTM. feel free to create the PR @FitseTLT

👋 @FitseTLT  do you have an estimated time of arrival for this PR? Thank you.

👋 @FitseTLT do you have an estimated time of arrival for this PR? Thank you.

Definitely today.

BugZero Checklist:

[x] [Contributor] Classify the bug:

Bug classification

Source of bug:

[ ] 1a. Result of the original design (eg. a case wasn't considered)
[x] 1b. Mistake during implementation
[ ] 1c. Backend bug
[ ] 1z. Other:

Where bug was reported:

[x] 2a. Reported on production
[ ] 2b. Reported on staging (deploy blocker)
[ ] 2c. Reported on both staging and production
[ ] 2d. Reported on a PR
[ ] 2z. Other:

Who reported the bug:

[ ] 3a. Expensify user
[ ] 3b. Expensify employee
[ ] 3c. Contributor
[ ] 3d. QA
[x] 3z. Other: Applause Internal Team

[x] [Contributor] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake.
Link to comment: https://github.com/Expensify/App/pull/39951#discussion_r1848253203

[ ] [Contributor] If the regression was CRITICAL (e.g. interrupts a core flow) A discussion in #expensify-open-source has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner.
Link to discussion:

[x] [Contributor] If it was decided to create a regression test for the bug, please propose the regression test steps using the template below to ensure the same bug will not reach production again.

Regression Test Proposal
Test:

Sign up with a new account.
Select "Manage my team´s expenses".
Complete the onboarding process selecting any option on the following steps.
Verify you land on Concierge chat and the conversation is auto scrolled to the bottom.

Do we agree 👍 or 👎

Payment Summary:

@FitseTLT paid $250 here (Contributor)
@brunovjk paid $250 here (Reviewer)


D435 for 3D motion tracking

|---------------------------------|------------------------------------------- |
| Camera Model                       | { D435 } |
| Firmware Version                   | {<IP_ADDRESS>} |
| Operating System & Version |   {Win10}  |
| Kernel Version (Linux Only)    |      |
| Platform                                 | {Laptop AsusX450}  |
| SDK Version                            |  { 2.16.5 }                          |
| Language                            |  {N.A}                          |
| Segment			|                          |
I have started to use the Intel RealSense SDK 2 technology for 3D motion recording, and would appreciate if you could help me with choosing an appropriate strategy. I used the device (D435) to record BAG files. The moving object was a mechanical device, shown bellow:

Now, I would like to generate a time history data of  X, Y, Z movements of the object. Since it's a multi (rigid) body with two arms a main hinge, motion data of only 3 points are enough for my analysis.
Could you guide me with choosing a suitable code/software to use for my analysis? I already did my search, but it wasn't conclusive.
Thank you,
Javad

If you want 3-dimensional XYZ values, the RealSense Viewer software has a 3D mode that can record 3D 'point cloud' data as a .ply format point cloud file instead of a bag.  This ply file can then be imported into other software packages if necessary for further editing.
For example, the ply can be imported into the MeshLab software, where the point cloud data could be converted into a solid mesh.
https://www.andreasjakl.com/capturing-3d-point-cloud-intel-realsense-converting-mesh-meshlab/
You can access point cloud mode in the RealSense Viewer by left-clicking on the '3D' button in the top corner of the Viewer.

Point clouds can also be generated and exported as a ply file using scripting if you are creating your own application.

Thank you MartyG. I already used the PLY files to create a 3D model of the mechanical device. I used MeshLab and SolidWorks to do it, and the 3D model was created successfully.
My current aim (and my question in this post) is different from 3D scanning and creating 3D models. I would like to record 10 seconds of a dynamic process (movements of the mechanical device). Then, X, Y, Z values of the motion of the mechanical device over time can be created. Something similar to the below graph. This graph is created using 2D video tracking, but I would like to add in-depth (Z) values to it as well:

Apologies for the delay in responding.  I am thinking that in this situation, putting marker images on the machine called ArUco markers might be appropriate.  The camera can then detect these markers (which can be simple flat paper print-outs) and track position and pose information.
There was an unofficial user-created extension of RealSense's Unity engine wrapper last year that had a demonstration of ArUco marker image tracking with RealSense and Unity.

Using markers would make it easy to track exactly the points of the machinery that you want to follow.
The unofficial RealSense Unity wrapper extension can be found below.  It is quite out of date now compared to the current official Unity wrapper, though the possibility exists of grafting the ArUco system of this old wrapper into the current official one.
If Unity is not appropriate for your project, you can also track ArUco markers with ROS.
http://wiki.ros.org/aruco

Hi J-mohajeri,
Another option to consider would be D435i. This unit combines the D435 with an IMU unit.
Please check out Slam with D435i example:
https://github.com/intel-ros/realsense/wiki/SLAM-with-D435i


lateinit property has not been initialized kotlin

I'm using your library in my project but I'm writing my code in Java not Kotlin and I'm using Butterknife for binding views when I'm trying to deal with your view in my code I'm getting this error
kotlin.UninitializedPropertyAccessException: lateinit property rectF has not been initialized
so I can't use setCornerRadius() method because rectF hasn't been initialized yet how can I use it??

Hello @aboulfotoh The problem is that the view has not been measured yet and the rectF used to delimit the boundary of the view hasn't been initialized.
I will try to fix it this weekend

all what you have to do is put the below line in your class CanvasRounder
private var rectF: RectF  = android.graphics.RectF(0f, 0f, 0f, 0f)
and it will work fine

Sound great, Give me a moment and I will fix it


Update App Name

The official name for the UC San Diego app is 'UC San Diego Mobile'
The App Store and Play Store app names will remain as 'UC San Diego'

This has been completed


Refactor ShaderParam using associated types

We could hide the link type entirely, simplifying the interface and streamlining the batch usage.

\o/

Hey, what happened?

Oh, nice!

Actually, with this in we no longer need to generate a typedef in the macros. One can just say RefBatch<YourStruct> and be good with it.

@kvark yeah, that's the end goal. I'm on it ;)

Merged now!


New purl for the Sample Description Vocabulary

Contact person name
Gulzaure Abdildina
Contact person email.
<EMAIL_ADDRESS>Overview in the README.md
Artifact name
Sample Description Vocabulary
Overview
Vocabulary for describing sample. A Sample is Physical System (typically a piece of material) composed by one or more Sample Components, exposed to the Instrument during a Measurement, typically after a Sample Preparation. Sample may be held by a Sample Holder and/or carried by a Sample Carrier during the Measurement.
Reach out
https://matwerk.datamanager.kit.edu/evoks/sampleDescription-1/de/
I would have some rules during the redirection of the URLs. I will write an email explaining them.
What is your artifact prefix?
sdv
Where is the artifact located on the Web (i.e. the URL)?
https://matwerk.datamanager.kit.edu/evoks/sampleDescription-1/de/
Do you want to enable content negotiation?

[ ] enable content negotiation

artifact type
HTML

Dear Said,
I would like to ask you to create a redirection rule. You have already done it for my first PIDA issue (https://github.com/Materials-Data-Science-and-Informatics/PIDA/blob/main/evoks/.htaccess).
The principle for this one is also the same. I want to change just the name of the vocabulary.
Redirect for evoks/sampleDescription terms (e.g. evoks/sampleDescription/aai)
RewriteRule ^MatWerkAcronyms/$  https://matwerk.datamanager.kit.edu/evoks/sampleDescription-1/en    [R=303,L]
RewriteRule ^MatWerkAcronyms/(.*)$  https://matwerk.datamanager.kit.edu/evoks/sampleDescription-1/en/page/$1    [R=303,L]
With best regards,
Gulzaure

Hi Said,
yes, I would like to create a new purl for sdv (sample description vocabulary). I wrote you also an email.
Best,
Gulzaure
From: Said Fathalla @.>
Reply to: Materials-Data-Science-and-Informatics/PIDA @.>
Date: Monday, 19. February 2024 at 12:31
To: Materials-Data-Science-and-Informatics/PIDA @.>
Cc: "Abdildina, Gulzaure (SCC)" @.>, Author @.***>
Subject: Re: [Materials-Data-Science-and-Informatics/PIDA] New purl for the Sample Description Vocabulary (Issue #52)
Hi Gulzaure,
I am a bit confused. do you want to change MatWerkAcronyms namespace to sdv? or create a new one for sdv in addition?
If you want to have a new one, then I'd expect the PID for aai would be
`https://purls.helmholtz-metadaten.de/evoks/sdv/aai
is that correct?
—
Reply to this email directly, view it on GitHubhttps://github.com/Materials-Data-Science-and-Informatics/PIDA/issues/52#issuecomment-1952255866, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A4TD4GAK6EXITXLYHY3KP6TYUMZYPAVCNFSM6AAAAABDAAYX2CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJSGI2TKOBWGY.
You are receiving this because you authored the thread.Message ID: @.***>

It works!
Thanks a lot, Said!
Best,
Gulzaure
From: Said Fathalla @.>
Reply to: Materials-Data-Science-and-Informatics/PIDA @.>
Date: Monday, 19. February 2024 at 12:31
To: Materials-Data-Science-and-Informatics/PIDA @.>
Cc: "Abdildina, Gulzaure (SCC)" @.>, Author @.***>
Subject: Re: [Materials-Data-Science-and-Informatics/PIDA] New purl for the Sample Description Vocabulary (Issue #52)
Hi Gulzaure,
I am a bit confused. do you want to change MatWerkAcronyms namespace to sdv? or create a new one for sdv in addition?
If you want to have a new one, then I'd expect the PID for aai would be
`https://purls.helmholtz-metadaten.de/evoks/sdv/aai
is that correct?
—
Reply to this email directly, view it on GitHubhttps://github.com/Materials-Data-Science-and-Informatics/PIDA/issues/52#issuecomment-1952255866, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A4TD4GAK6EXITXLYHY3KP6TYUMZYPAVCNFSM6AAAAABDAAYX2CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJSGI2TKOBWGY.
You are receiving this because you authored the thread.Message ID: @.***>

Hi Said,
How are you? I saw that you are going to present your work at the HMC Conference. Great to hear that!
I wanted to ask if you could please change the vocabulary name on the link from issue #52. https://matwerk.datamanager.kit.edu/evoks/sampleDescription-1/en Could you please to change the vocabulary name in links from “SampleDescription-1” to “SampleDescriptionVocabulary-1”.
Thanks a lot and best,
Gulzaure
From: Said Fathalla @.>
Reply to: Materials-Data-Science-and-Informatics/PIDA @.>
Date: Tuesday, 20. February 2024 at 11:07
To: Materials-Data-Science-and-Informatics/PIDA @.>
Cc: "Abdildina, Gulzaure (SCC)" @.>, Author @.***>
Subject: Re: [Materials-Data-Science-and-Informatics/PIDA] New purl for the Sample Description Vocabulary (Issue #52)
Closed #52https://github.com/Materials-Data-Science-and-Informatics/PIDA/issues/52 as completed.
—
Reply to this email directly, view it on GitHubhttps://github.com/Materials-Data-Science-and-Informatics/PIDA/issues/52#event-11859451577, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A4TD4GBAO6274EOKIH3A4UTYURYVLAVCNFSM6AAAAABDAAYX2CVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJRHA2TSNBVGE2TONY.
You are receiving this because you authored the thread.Message ID: @.***>

Super! It works. Thanks a lot!
From: Said Fathalla @.>
Reply to: Materials-Data-Science-and-Informatics/PIDA @.>
Date: Tuesday, 22. October 2024 at 17:00
To: Materials-Data-Science-and-Informatics/PIDA @.>
Cc: "Abdildina, Gulzaure (SCC)" @.>, Author @.***>
Subject: Re: [Materials-Data-Science-and-Informatics/PIDA] New purl for the Sample Description Vocabulary (Issue #52)
Hi Gulzaure,
I am doing well. Thanks!
I changed it. Could you please check?
—
Reply to this email directly, view it on GitHubhttps://github.com/Materials-Data-Science-and-Informatics/PIDA/issues/52#issuecomment-2429529057, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A4TD4GEIS33AOI2OJX6FGMDZ4ZSBRAVCNFSM6AAAAABQL66PBOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMRZGUZDSMBVG4.
You are receiving this because you authored the thread.Message ID: @.***>


Titles and original problem statement are in plain text

The "title" on index.html and the original problem statement are just plain text. They should be math.
@sueannma : the data file should be modified to use $$'s around some latex for the math. E.g., $$7x+13=1$$. For more advanced constructs, type the expression into mathlive.io and then copy the latex from that. E.g., should the "-1/6" in the first problem really use a "/" or should it be a 2D fraction (1 over 6)? If the later, then it is \frac{1}{6}.
@BenetechSchwab: you probably need to call MathLive.renderMathInDocument(); somewhere on the index.html page.

agreed - see issue #35

Fixed


Align client classes with the TSP specification

Fixes #3
Signed-off-by: Geneviève Bastien<EMAIL_ADDRESS>

It matches the master of the trace-server-protocol as of today. It will be a breaking change for the trace extension and will need a matching patch in the trace server. I'm working on it, once the builds are fixed there!

Thanks for the review, I'll merge when all the components are ready to merge (the theia-trace-extension one will need to be updated once this is in to link to the new next version of this packages)


Hello world example.

Please create a simple hello world example.
I cant figure out how to use gui for a simple hello world program because I don't understand how to correctly build anything with Zig. Please create a single simple hello-world so I can get started. Thanks.

This is supposed to be the "hello world" examples, but I see I didn't explicitly say to run the command.  Does running zig build run-standalone-sdl work for you?

Your examples work perfectly thank you. I'm just confused trying to make any library work with zig. Sorry, TMI all at once with zig.
Looking over your build.zip, it makes more sense to me now.
I think what I will do for my app: Using a copy of your example, put your gui code in src/gui, my code in src/app, and edit build.zip so the paths are corrected.
Again, thank you. Kind regards, joe budd

No problem.  That plan sounds good to me.
It can be difficult right now because zig is changing so fast.  If you have more issues let me know!


Let assign ALIAS to Select Columns

Hi
If you need this query:
Select a.name, b.name from section as a left join gallery as b on(a.idsection=b.section_idsection)
In the Medoo format:
$Data = $database->select("section",
[
"[>]gallery" => ["idsection" => "section_idsection"]
],
[
"section.name",
"gallery.name",
]
]);
But you can't differentiate between NAME of Section or NAME of GALLERY. You need have an ALIAS in the query. So:

Add in the LINE: 127
 protected function column_quote_as($string)
 {
         return str_replace('.', '`.`', $string);
 }

Add in the LINE: 405 (original line)

if (is_array($columns)){
foreach($columns AS $Columns){
$columnsTemp =explode('[>]',$Columns);
$columnsVal = ($columnsTemp[1]!='') ? '' . $columnsTemp[0] . ' AS '' . $columnsTemp[1] . ''':'' .$columnsTemp[0] . '';
$ColumnsEnd[] = $columnsVal;
}
}else{
$ColumnsEnd[]='' .$columns . '';
}

Change the LINE: 408 (original line)
 			is_array($ColumnsEnd) ? $this->column_quote_as( implode(', ', $ColumnsEnd) ) :

And now, the new format to take the query is:
$Data = $database->select("section",
[
"[>]gallery" => ["idsection" => "section_idsection"]
],
[
"section.name",
"gallery.name[>]NameGallery",
]
]);

How to convert following query to  medoo select ?
select *, date_format(column_name, '%d %b %Y') as column_alias from table_name;


Translation is not shown if there is no direct translation in EMBEDDEDMAP

Fixed here: https://github.com/OrienteerDW/Orienteer/commit/3fd854c6a9bb3c322b0ec520521ab2791ce81ae3


[platform] platform independent spinel radio

This change refactors radio-spinel into a platform independent
header-only library so that other platforms (baremetal/RTOS) can enable
Thread with RCP mode as well.

@jwhui @bukepo @abtink
I'm using template parameter to provide a static polymorphism. Would you prefer this style or using a  base class with virtual functions?

@jwhui @bukepo @abtink
I'm using template parameter to provide a static polymorphism. Would you prefer this style or using a base class with virtual functions?

In general, we should avoid virtual functions especially if we only intend to include a single instance at a time.

@gjc13 To make openthread-spinel-ncp a library, we need to export src/lib/spinel as its public includes.

@gjc13 To make openthread-spinel-ncp a library, we need to export src/lib/spinel as its public includes.

Since we are exporting src/lib, I'd expect users to include spinel/xxxx.h.

@bukepo @abtink Friendly reminder :)

When updating the comments I found some extra dependency on posix timeval struct definition.
Updated code to remove this dependency.


added I2C driver for TeraRanger One

also see PR PX4/DriverFramework#52

sorry, I'm a little confused, does the trone.cpp in drivers directory not working right now? I thought that's also I2C?

This is based on the new DriverFramework platform,  so this driver should work on all platforms like RPi, Snapdragon, NuttX, etc.

@mhkabir It won't work on NuttX just yet but sometime in the future we might be able to merge the parallel driver worlds.

Reviewed this and https://github.com/PX4/DriverFramework/pull/52. Have a look at the comments.

@julianoes Can you manage the integration with Nicolas?

All looked, good, thanks! Rebased to master.


scrollController of listView not working in singleChildScrollView

What I want:  get the scrolling position in order to make refresh and loadMore.
scrollerController only work at 1, but I hope it can work at 2
the controller's position is the only different.

working

singleChildScrollView
  controller: _scrollController
  --> column --> column ...  --> ... container
  --> ListView.builder
  shrinkWrap: true,
  physics: ScrollPhysics(),

not working

singleChildScrollView
  --> column --> column ... --> ... container
  --> ListView.builder
  controller: _scrollController,
  shrinkWrap: true,
  physics: ScrollPhysics(),

scrollerController only doing the following printing.
_scrollController.addListener(() {
  print('bbb');
});

there are many widgets between, so I put singleChildScrollView and ListView in different files.
Is this result is a issue or by design ? Will it fit or change in the future ?
[✓] Flutter (Channel stable, v1.12.13+hotfix.5, on Mac OS X 10.15.2 19C57, locale en-US)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 11.3)
[✓] Android Studio (version 3.5)
[!] IntelliJ IDEA Ultimate Edition (version 2019.1)
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
[✓] Connected device (1 available)

! Doctor found issues in 1 category.

Hi @chengweiou
does this code  represent the issue you are describing?
thank you

Hi, @iapicca
Thats the printing that I want to see. but they still some different.

It should not be 2 separated scroll.
I put them in different files, I will check when we finish this one

this is the code shows the current problem. I copy from your code.

Hi @chengweiou
I'm not anymore sure about
what you are trying to achieve,
in your example the ScrollController of the ListView
is not "firing" simply because the listview isn't scrolling

sample
import 'package:flutter/material.dart';

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

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) => MaterialApp(
    home: MyHomePage(),
  );
}

class MyHomePage extends StatefulWidget {
  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {

  static ScrollController _listViewController;

  @override
  void initState() {
    _listViewController  = ScrollController()
      ..addListener(()=> print('list view is scrolling'));
    super.initState();
  }

  @override
  void dispose() {
    _listViewController?.dispose();
    super.dispose();
  }

  @override
  Widget build(BuildContext context) => Scaffold(
    body: ListView(
      controller: _listViewController,
      children: <Widget>[
        Container(
            color: Colors.yellow,
            height: MediaQuery.of(context).size.height / 3,
          ),
        ... <Widget>[for (int i = 0; i < 100; ++i) Center(
            child: Text('$i', textScaleFactor: 3,)
        ),],
      ],
  ),);
}

Hi @iapicca
sorry, your sample is not the code that I want.
I'm trying to use controller inside instead of outside because they are not in one file
i update my code and add more comment. Hope this can help
thank you

Hi @chengweiou
I apologize if my answer was confusing or inaccurate.
This platform is not meant for assistance on personal code,
but to improve or fix bug within flutter,
in your case the ScrollController is working as intended
and is not triggering simply because is not scrolling.
If you need assistance to implement your code
you may get some help if you post it on Stack Overflow.
Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
Thank you

What I want: get the scrolling position in order to make refresh and loadMore.
scrollerController only work at 1, but I hope it can work at 2
the controller's position is the only different.

working

singleChildScrollView
  controller: _scrollController
  --> column --> column ...  --> ... container
  --> ListView.builder
  shrinkWrap: true,
  physics: ScrollPhysics(),

not working

singleChildScrollView
  --> column --> column ... --> ... container
  --> ListView.builder
  controller: _scrollController,
  shrinkWrap: true,
  physics: ScrollPhysics(),

scrollerController only doing the following printing.
_scrollController.addListener(() {
  print('bbb');
});

there are many widgets between, so I put singleChildScrollView and ListView in different files.
Is this result is a issue or by design ? Will it fit or change in the future ?
[✓] Flutter (Channel stable, v1.12.13+hotfix.5, on Mac OS X 10.15.2 19C57, locale en-US)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 11.3)
[✓] Android Studio (version 3.5)
[!] IntelliJ IDEA Ultimate Edition (version 2019.1)
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
[✓] Connected device (1 available)

! Doctor found issues in 1 category.

Did you find the solution? am facing the same issue

Hi @braysonjohn148
the scrollController is behaving as intended,
please look the samples provided in the comments above
Thank you


Misleading error message when an image has an incorrect path

Steps to reproduce:

Create a file called docs/pages/test.png in a content directory within your gravitational/docs clone.
Add the following content to the page:

 ---
 title: test
 description: test
 ---

 This is an image:

 ![my image](../fake.png)

Start the dev server via yarn dev and navigate to localhost:3000/docs/test

The error is:
Error: Image with src "../fake.png" is missing required "width" property.

For a docs author who isn't familiar with this error, it can be difficult to debug. We should stat the file path of an image first to make sure it exists, and if not, show a more descriptive error.

Need to check if the misleading error message still displays on the Docusaurus website.


Updated liche.

Hopefully improves https://github.com/thanos-io/thanos/issues/3060

Wow, so many links are broken now cc @thisisobate
I think it's worth to spend some time in adjusting our script for creating links ):

Wow, so many links are broken now cc @thisisobate
I think it's worth to spend some time in adjusting our script for creating links ):

wow...would look into this now

The problem is that probably our liche script should check if the links will work on master website, not on master :thinking:

okay...looking into this

but was it at least faster?

Yup

Will be part of https://github.com/thanos-io/thanos/pull/3067/


Port CoreFx PR 38271: Fix Statement Command Cancellation (Managed SNI)

Ports dotnet/corefx#38271 by @Wraith2 to fix #109
(The PR is copy of the original PR, so we can review these changes first here)

@Wraith2 this PR and the PR dotnet/corefx#38271 have same changes but the newly added tests are failing on Linux from this PR. Could you take a look?
Manual Tests Log: log_15_16802.zip

The range failing sort of makes sense. There's clearly some difference in this repo to corefx to do with cancellation or https://github.com/dotnet/SqlClient/pull/234 wouldn't have been needed. I think that PR blocks this PR from being effective because you can't get to this fix if the other blocks it and this is higher in the stack. So I'd advise queuing this one after the other and retesting once it's merged.
For netfx the test was never intended for that build. A value of .99 means it doesn't exhibit the problem that this PR is attempting to fix though which is probably good. I think it also means the timing is slightly wonky since it's supposed to cancel after 1 second and less than a second has elapsed, we could probably just relax the lower bound because it's the upper bound that signals the error condition.

There's now a lock that is optionally entered. I'm curious how that can possibly be safe? If entering the lock fails then execution simply continues without a lock held.

It's safe because there are only two ways to aquire it and one of them is the attention packet setup and send which doesn't mutate local state. If you're attempting to send the attention packet and the lock is already held you need to ignore the lock because even though something else is holding it you know you're not going to mutate and your very purpose is to cancel the thing that's holding the lock. If you're attempting to run and the lock is held then you need to wait for it.

OK, so it is a requirement, that the lock is held but it does not matter by whom. Could it not happen that the lock is held, so taking the lock is skipped, and then immediately the lock is released by the code? That would allow execution without any lock held.

Yes, The only order that this can happen in is that an execute can release the lock while an attention is also in flight and then a second execute can start, that shouldnt' cause a problem because the attention doesn't really mutate, though it will probably set the attention sent flag.
Do you have a suggestion on a better way to approach this? using locks in code shared between sync and async executions is a bit dangerous in my opinion anyway. What would meet the requirements in this case is a counted mutex object that you can try enter based on the current value, execute could only enter if it is <1 and attention can only enter if it's <2, i don't think the exeisting mutexes support this behaviour.

@Wraith2 I am totally unqualified to give advice on this :smile:. I don't know how the codebase works.
I have learned over the years that holding a lock for an unbounded amount of time (such as during IO) is usually an impossible design. Doing this locks out other access to the system/data (such as cancellation). My experience is that this creates functional problems that cannot be fully fixed under such a design.
Often, a way out of this is to use a state machine approach. For example, track ongoing and queued operations in a data structure. Each access of those structures happens under a lock within microseconds. When an IO/operation completes it must check those structures and possible update them or initiate other work. I hope this does not sound too vague.
My understanding is that ADO.NET locks while doing IO. I have seen a number of problems on this issue tracker around cancellation, concurrency and locking. I got the impression that the concurrency design is not fundamentally correct. But this might be naive given that I am not familiar with the codebase.

in general I agree but I'm not a position to totally rewrite the internals with lock free mechanisms at the moment. I'll put that on the list of things to consider.

@cheenamalhotra Afaiu this issue exists also on 1.1.x version and that why on EF Core 3.1. Any plans to port this fix to there too?

@olljanat You can just update to use 2.1.0 with EF Core 3.x: https://erikej.github.io/efcore/sqlclient/2020/03/22/update_mds.html


chore: import repository https://github.com/adithya180498/-node-http-watch-pipeline-activity.git

this commit will trigger a pipeline to generate the CI/CD configuration which will create a second commit on this Pull Request before it auto merges

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign
You can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign
You can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

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

Test name
Commit
Details
Rerun command

verify
8887ea45628489f2f9e86bf0d1d4160d42f722c7
link
/test verify

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


Fix the FlinkRunner for Dataproc

Fix the dataproc Flink version
Make sure the Flink cluster has the external IPs.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

To make the notebook running fine with Dataproc Flink, we also need to do this:
from apache_beam.options.pipeline_options import DebugOptions
options.view_as(DebugOptions).add_experiment('disable_logging_submission_environment')

@tvalentyn @damccorm FYI


[MXNET-1305] Scala interpreter instructions

Description
Add instructions to the scala_setup page for using the interpreter and link to in the only example featuring the interpreter. It also adds a note that we only support scala 2.11. This should address the problem seen in #13468.
@lanking520 @piyushghai @andrewfayres @aaronmarkham
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

@zachgk Thanks for the cotribution!
@mxnet-label-bot add [pr-awaiting-review, Scala]
@lanking520 @piyushghai @andrewfayres
Could you please help review this PR?


Clarifying it must be a smart query

It wasn't very clear that it i needed to attach the subscription to a smart query.

Thanks!


Update model diagram

Changes:

Added plantuml github actions

Please help me check if this model is correct 😃


Kbin.social account not working

Hey,
I created a kbin.social account but when trying to start the bot I get an error message:

[a@eos Botit]$ npm start

<EMAIL_ADDRESS>start
node dist/scheduler.js

[ERROR] [LEMMY] Error while trying to login, host might be down
[ERROR] [SCHEDULER] An error occurred: Error: FetchError: invalid json response body at https://kbin.social/api/v3/user/login reason: Unexpected token '<', "<!DOCTYPE "... is not valid JSON

Is it supposed to work with kbin accounts?
Cheers

Hello! It doesn't work with kbin accounts yet since kbin doesn't have an API (for now).
But you should be able to use any lemmy account, just make sure you don't mark that account as "bot account" on lemmy, that will make the posts not show up on kbin.
Let me know if you have any other questions or suggestions!

Hey! Thanks for the reply.
That's unfortunate. I have to think about it again then since I made the decision to not support lemmy directly. I am not in line with their political believes at all and don't want to support that platform even indirectly.

I understand, as far as I know there are other instances that are run by different admins all together, since it's open source everyone can fire up a lemmy instance.
Maybe if you find one you could use it.
Thanks for the comments!

Yeah I might do that. Thanks
Case closed anyways. :)


perl not compiling

Hi,
I'm trying to compile perl on
wget http://www.cpan.org/src/5.0/perl-5.24.0.tar.gz
tar xvzf perl-5.24.0.tar.gz
cd perl-5.24.0/
./Configure -d
make

It fails at a step where it tries to load Pod::Simple::SimpleTree, but it seems to not be in miniperls' INC PATH. It would seem that some early stage in the build process fails due to someting in WSL not working as expected.

PERL5LIB=/path/to/perl-5.24.0/cpan/Pod-Simple/lib make

The strange thing is that it isn't picking it up automatically (on Linux it does).
This makes utilities like plenv and perlbrew not work correctly (they don't set the PERL5LIB, and they use a temporary directory for compiling perl).
Maybe @xsawyerx can help out or point to someone that can help find out what's going on

@pplu It depends where the environment variable is defined.  Could you check in your ~/.profile file for "PERL5LIB"?
If it's present in profile you'll have to launch bash.exe as a login shell to get it to load automatically:
bash.exe --login

I might be misunderstanding this problem, but you shouldn't have to set the PERL5LIB environment variable in order to compile perl.
For any perl + Windows debugging, I would personally refer to @bulk88. He's the expert.

@benhillis I coincide with SawyerX. It looks like somewhere in the compilation chain, that PERL5LIB env variable gets lost. It doesn't on Linux.
@bulk88: I found specially helpful this GitHub bug https://github.com/kraih/mojo/issues/954 and where it links to, to get the the subsystem installed and working, and some Perl issues corrected

Anyone figured out how to get perl compilation working?

@erossetto: I've got perl to compile manually (no plenv) from a source tarball by setting the PERL5LIB as @lateau suggests. It's an ugly workaround, but it works 😄

The problem is that File::Find can't search recursively on the FUSE filesystem that is used by BashOnWindows. See also #186.
This will make the compilation work:
echo "dont_use_nlink='define'" >> Policy.sh
./Configure -des
make
make install

#186 is marked as fixed and closed, so doing the same here.  Please reopen if you are still hitting issues and we can investigate.

@stehufntdev I've been able to compile Perl 5.24.0 😄


Package conflict with pandas parqet engine in astrophysics default image

From the astrophysics default image running the light curve classifier notebook, there is an import that conflicts with pandas such that it can't read parquet files.  This same error does not happen when starting with the astrophysics default image from scratch without the list of imports.  Error below:
`---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
Cell In[3], line 9
5 gdd.download_file_from_google_drive(file_id='13RiPODiz2kI8j1OKpP1vfh6ByIUNsKEz',
6                                     dest_path='./data/df_lc_458sample.parquet',
7                                     unzip=True)
8 import pandas as pd
----> 9 df_lc = pd.read_parquet("./data/df_lc_458sample.parquet")
11 #get rid of indices set in the light curve code and reset them as needed before sktime algorithms
12 df_lc = df_lc.reset_index()
File /opt/conda/lib/python3.10/site-packages/pandas/io/parquet.py:654, in read_parquet(path, engine, columns, storage_options, use_nullable_dtypes, dtype_backend, filesystem, filters, **kwargs)
501 @doc(storage_options=_shared_docs["storage_options"])
502 def read_parquet(
503     path: FilePath | ReadBuffer[bytes],
(...)
511     **kwargs,
512 ) -> DataFrame:
513     """
514     Load a parquet object from the file path, returning a DataFrame.
515
(...)
651     1    4    9
652     """
--> 654     impl = get_engine(engine)
656     if use_nullable_dtypes is not lib.no_default:
657         msg = (
658             "The argument 'use_nullable_dtypes' is deprecated and will be removed "
659             "in a future version."
660         )
File /opt/conda/lib/python3.10/site-packages/pandas/io/parquet.py:66, in get_engine(engine)
63         except ImportError as err:
64             error_msgs += "\n - " + str(err)
---> 66     raise ImportError(
67         "Unable to find a usable engine; "
68         "tried using: 'pyarrow', 'fastparquet'.\n"
69         "A suitable version of "
70         "pyarrow or fastparquet is required for parquet "
71         "support.\n"
72         "Trying to import the above resulted in these errors:"
73         f"{error_msgs}"
74     )
76 if engine == "pyarrow":
77     return PyArrowImpl()
ImportError: Unable to find a usable engine; tried using: 'pyarrow', 'fastparquet'.
A suitable version of pyarrow or fastparquet is required for parquet support.
Trying to import the above resulted in these errors:

Missing optional dependency 'pyarrow'. pyarrow is required for parquet support. Use pip or conda to install pyarrow.
Missing optional dependency 'fastparquet'. fastparquet is required for parquet support. Use pip or conda to install fastparquet.`

Hi @brianppowell, thanks for reporting. I was unable to reproduce this error using the astrophysics default image. The error seems to indicate that neither pyarrow nor fastparquet are installed. Both kernels that are available for me (root and science_demo) have pyarrow installed and were able to execute the relevant notebook cells. Do you know which kernel you were using to run the notebook?

Thanks for following up, @troyraen.  Here is the kernel info:
(science_demo) jovyan@jupyter-bpowel:~$ cat /proc/version
Linux version 5.10.184-175.749.amzn2.x86_64 (mockbuild@ip-10-0-46-190) (gcc10-gcc (GCC) 10.4.1 20221124 (Red Hat 10.4.0-1), GNU ld version 2.35.2-9.amzn2.0.1) #1 SMP Wed Jul 12 18:40:28 UTC 2023

Thanks, from that I see that you're in the science_demo environment and working on the command line. Are you copying code out of the notebook and running it on the command line? If so, that's fine and should also work (I often do it this way), but now it's not clear to me what steps you're taking that result in this error. I'm also not sure what you mean by this line:

This same error does not happen when starting with the astrophysics default image from scratch without the list of imports.

Can you post a reproducible example with all of your steps?

The image has been updated since March, so I would go ahead and close this as outdated. Please open a new issue if you still run into this problem along with a reproducible example so we can track down the problem. Thanks!


KTH dataset content question

Hi, I use gdown --fuzzy https://drive.google.com/file/d/1d2UfHV6RhSrwdDAlCFY3GymtFPpmh_8X/view?usp=sharing to download KTH64 dataset. When I run the code dataset, test_dataset = get_dataset(DATA_PATH, config, video_frames_pred=config.data.num_frames) I got an KeyError.

I found that the file persons.pkl is empty (5 bytes and only {}). I would like to confirm with you about the dataset if it is all right. Thanks!

I'm not sure why gdown fail, but the error says to big and its indeed very big; since it doesn't work, you can instead download it manually by following the link on your browser, its 13Gb.
https://drive.google.com/u/0/uc?id=1-R_srAOy5ZcylGXVernqE4WLCe6N4_wq&export=download


Assests page not displaying most up-to-date value after refresh

A user report :
"Opened https://app.osmosis.zone/assets and noticed my value looked surprisingly low, so I routed to https://legacy.osmosis.zone/assets to get better visibility. I refreshed both pages. The new front-end is not reflecting the latest value for pool 611, which I added to my bond total this morning. Even after refresh, it will not update the value, and in fact they are showing different values for every single one of my pools. Going INTO the pool page itself, the number of GAMM tokens are identical, but the value is not being updated in the NEW UI - for my understanding, the Legacy shows the correct value.
Keep in mind this is after clicking into the pool, waiting, then refreshing cache."

All assets should have price info properly configured


Manage Navigating event on ElectronNET implementation

Manage Navigating event on ElectronNET implementation:

[ ] On regular navigation
[ ] On iframe navigation (in order to do the same behavior like on BlazorMobile)

If we can take the same route as on GeckoView WebExtensions, here is the doc to register WebExtensions on ElectronJS: https://electronjs.org/docs/api/browser-window#browserwindowaddextensionpath

It seem that actual ElectronNET implementation does not support WebExtension method yet as it seem not exposed.
If not integrated by developpers, i will try if i can do a dev about this and pull request...

Just leaving an information.
If we have to locate where is located the WebExtension path, we may look about the Entry point  path of the ASP.NET Core Webservice executing.
This should work:
AppDomain.BaseDirectory

https://docs.microsoft.com/fr-fr/dotnet/api/system.appdomain.basedirectory?view=netframework-4.8

Done


New August update broke xDebug listener on port 9003 (Laravel Sail)

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.82
OS Version: macOS 13.5.2

Steps to Reproduce:

Update to the latest vscode version
Create a new Laravel Sail project (https://laravel.com/docs/10.x/installation#laravel-and-docker):
curl -s https://laravel.build/example-app | bash
cd example-app
edit .env and set SAIL_XDEBUG_MODE=develop,debug,coverage
Add vscode launch configuration:
create new ".vscode" folder in the project root, create file "launch.json" in the new folder, place contents:
{  "version": "0.2.0",  "configurations": [      {          "name": "Listen for Sail Xdebug",          "type": "php",          "request": "launch",          "port": 9003,          "log": true,          "pathMappings": {              "/var/www/html": "${workspaceFolder}"          },          "hostname": "localhost",          "ignore": ["**/vendor/**/*.php"],          "xdebugSettings": {              "max_data": 65536,              "show_hidden": 1,              "max_children": 100,              "max_depth": 3          }      },  ] }
Start Laravel Sail:
./vendor/bin/sail up -d
Set a break-point somewhere and try to execute or use artisan tinker
Laravel will output that it can't connect to xDebug client:
Could not connect to debugging client. Tried: host.docker.internal:9003 (through xdebug.client_host/xdebug.client_port).

I think it has something to do with the new Built-in port forwarding feature..
I'll just add that it doesn't work for me, mainly on a project in progress, and it worked until yesterday... Then vscode updated itself and it no longer works. If I downgrade the version to July (1.81) it works again.

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.82.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Perhaps related to the DNS change that came with Node 18. See https://code.visualstudio.com/updates/v1_82#_update-highlights-for-nodejs
I suggest you report the issue to the debug extension author.

Reported this to the author of that plugin, thank you.
https://github.com/xdebug/vscode-php-debug/issues/922

Please edit the link in your previous comment to it goes to the issue. That way the extension author should see a link back to here.


Update spec file usage

Use chain spec files with explicit network naming.

@orriin please update this with a link to the parent PR


VisIt not rendering veryclosetogether particles

The particle positions only differ after the 8th signficant digit,data supplied in double-precision. Checking actual values for the vtkPoints throughout VisIt's pipeline, they appear to be passed correctly without losing precision, so the issue is probably in the rendering. (which may require a vtk update). I have provided a sample dataset given by the customer (Will Schill). Create a Pseudocolor plot of mass.Zoom in on the red dot upper left corner.Changing point type to icosohedron or other glyphed option doesn't help.
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 1134
Status: Pending
Project: VisIt
Tracker: Bug
Priority: High
Subject: VisIt not rendering veryclosetogether particles
Assigned to: -
Category: -
Target version: -
Author: Kathleen Biagas
Start: 07/19/2012
Due date:
% Done: 0%
Estimated time:
Created: 07/19/2012 01:39 pm
Updated: 08/03/2012 05:50 pm
Likelihood: 3 - Occasional
Severity: 4 - Crash / Wrong Results
Found in version: 2.5.1
Impact:
Expected Use:
OS: All
Support Group: Any
Description:
The particle positions only differ after the 8th signficant digit,data supplied in double-precision. Checking actual values for the vtkPoints throughout VisIt's pipeline, they appear to be passed correctly without losing precision, so the issue is probably in the rendering. (which may require a vtk update). I have provided a sample dataset given by the customer (Will Schill). Create a Pseudocolor plot of mass.Zoom in on the red dot upper left corner.Changing point type to icosohedron or other glyphed option doesn't help.
Comments:
Added a vtk formatted file of only the 'fragments' part of the .sv file. It has 100 points.With this file it looks like the 3d view gets very messed up. I can rotate and see the particles lying in a plane, but VisIt's bounding box doesn't surround them. Projecting to 2D with XAxis Cartesian (or YAxis Cartesian) helps to see more of the particles.

1142_frag.vtk.zip
1142_fragment_cloud.sv.zip


Bump patch version

Codecov Report

Merging #75 (002ea29) into main (4638a1d) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #75   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           10        10
  Lines          316       316
  Branches        40        40
=========================================
  Hits           316       316

Continue to review full report at Codecov.

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


Check and update README to allow any UID

Any UID should be allowed and working fine, so people can use --user 123456 or --user $UID if they so desire.
Update also README to suggest using a random UID in [2000-32000].

I tried using user: "1000" in docker-compose and I got this in the logs,
runuser: may not be used by non-root users
runuser: may not be used by non-root users
runuser: may not be used by non-root users
runuser: may not be used by non-root users
runuser: may not be used by non-root users
runuser: may not be used by non-root users
runuser: may not be used by non-root users
runuser: may not be used by non-root users
runuser: may not be used by non-root users
runuser: may not be used by non-root users
runuser: may not be used by non-root users
runuser: may not be used by non-root users
runuser: may not be used by non-root users
runuser: may not be used by non-root users
And it failed to start.

Did you pull the latest version? Which tag?

wernight/plex-media-server:autoupdate
I do have another step that pulls the latest version.
Thanks :)

The "any UID" is only for the non-autoupdate. This is because autoupdate needs to install Plex, so unless I remove any permission on the Linux FS, the user in the contains needs to be root. After that it changes to plex random UID.
If you feel the documentation isn't clear, what would you suggest?

Sorry I'm a bit confused, you say it runs as root and then changes to a random uid... Why can't it run as root and then change to the ID picked through --user $UID?

autoupdate starts as root inside the container to install/upgrade plex, then runs as UID 797 (which was randomly chosen).


Error pulling the dataset

I got this error when I ran the code showed in the readme file:
azure.core.exceptions.ClientAuthenticationError: Server failed to authenticate the request. Please refer to the information in the www-authenticate header.
RequestId:1a7f1721-c01e-000b-2e71-a033f4000000
Time:2022-07-25T21:59:04.0285186Z
ErrorCode:NoAuthenticationInformation
Content: <?xml version="1.0" encoding="utf-8"?><Error>NoAuthenticationInformation<Message>Server failed to authenticate the request. Please refer to the information in the www-authenticate header.
RequestId:1a7f1721-c01e-000b-2e71-a033f4000000
Time:2022-07-25T21:59:04.0285186Z</Message></Error>

I was able to download directly the zip file (not using azure-storage-blob lib).


Support for zvuk.com

Thank you for the extension. Please add support also for Zvuk, because it seems like audio ads there cannot be blocked in other ways.

Hello!
Unfortunately, it looks like the website only allows registration from specific countries, and I'm not in one of them.
I could use a VPN to bypass that, but their registration form only allows phone numbers from those countries as well.
But even if managed to get a free account (e.g. by donation), it would last for 30 days. After that, I'd have to get another free account to make sure the website continues working. So, basically, I'd need to have a stash of free accounts.
For the reasons above, I don't think this would be a sustainable endeavor for me, sorry.

@guihkx Registration by an e-mail address is possible there:

Oh, I missed that. Thanks for letting me know, I'll have a look.


Fix generic tests returning "TypeError: 'NoneType' object is not subs…

…criptable"
Generic tests return an error "TypeError: 'NoneType' object is not subscriptable", because they are listed in the nodes, but not relevant for dbt-coverage.

relates to https://github.com/slidoapp/dbt-coverage/issues/52


does not work when video speed is changed

If you use Youtube and change the video speed the soundfixer changes speed back to 1x. It does not work with a different playback speed.

duplicate of #2. This is a known issue, not sure it's possible to solve.


Upgrade & re-test latest Tantivy and Lucene

There have been a number of search-time optimizations inspired by Tantivy folded into Lucene lately.  And I'm sure Tantivy is making improvements too!  Let's re-upgrade to the latest & greatest and re-test?

Will try to get this upgraded today.

Comparing the new change (left) to the prod website (right).
Same setup used in test as in the prod website:

Ran on m5.12xlarge machine
JDK 20
Rust 1.71.1

Lucene has improved in the benchmark between 9.7->9.8 with COUNT (-16%) and TOP_100 (-14%)

Results.json from results above.
results.json


WCAG 1.4.3: (ExpenseItDemo/ Label "Total Expenses ($)" and its value label in 'Create Expense Report' view and "View chart Window" view) [High Contrast Black] Contrast ratio failure for large text.

The following accessibility issue needs investigation.
App: ExpenseItDemo
Element path: Label "Total Expenses ($)" and its value label in 'Create Expense Report' view and "View chart Window" view
Issue Details: Contrast ratio failure for large text. Details:
Font Size = 18
Font Weight = regular
How To Fix: https://www.w3.org/TR/UNDERSTANDING-WCAG20/visual-audio-contrast-contrast.html
This accessibility issue was found using Accessibility Insights for Windows, a tool that helps debug and find accessibility issues earlier. Get more information and download this tool at https://aka.ms/AccessibilityInsights.