Disabled-worlds 🆘

I found small bug, when I turn off the scoreboard in a world where I have a mini-game plugins with a built-in scoreboard, it breaks the built-in scoreboard and the scoreboard in the mini-game flashes and does not want to show up.this problem is also with other minigames with scoreboard (the scoreboard is turned off in the minigame world), please need help with this. I tack link to video where you can see what's going on. I hope it can be solved somehow and I'm sorry for my bad English ...
Video link to a demonstration of the problem: https://streamable.com/mi0ywe

No problem. I am not also a native english speaker.
Can you send me your LOG file for RealScoreboard, config and plugin version?

I'm sending you a log when the server is loading, everything where the name or something associated with it occurs:
[17:26:24] [Server thread/INFO]: [RealScoreboard] Loading RealScoreboard v11-4-2020 Build 1
[17:26:18] [Server thread/WARN]: Plugin RealScoreboard v11-4-2020 Build 1 does not specify an api-version.
[17:26:48] [Server thread/INFO]: ------------------- RealScoreboard -------------------Checking the server version.Your server is compatible with RealScoreboard!Setting up the configuration.Your config version is: 3Your config file is updated to the latest version.Registering Events.Registering Commands.Starting up the Scoreboard.Loaded sucessfully.Server version: v1_14_R1If you have any questions or need support, feel free to message JoseGamer_PThttps://www.spigotmc.org/members/josegamer_pt.40267/------------------- RealScoreboard
I also send the plugin and config from it.
Config: https://pastebin.com/KqZVXckC
plugin from server: https://drive.google.com/open?id=1qfZJnqride1jiHge5j51vcMlTuTgzodB

if you still need to send something then just say so.


timePicker should support 'seconds' as well

Currently, timePicker only shows hours and minutes. Please add support for the seconds.

@yurigenin, this is already in master under #4768.  As I'm sure you can imagine, it takes a tremendous amount of effort to maintain a very popular OSS repository.  As such, please extend us the courtesy of searching both open and closed issues before opening a new one - especially for feature requests.
If you would like to partake of the sheer awesomeness of timepicker seconds, please feel free to pull down master and run npm install followed by grunt.  The built files (including the new docs) will be in the dist folder.


Server builder plugin and server reflection

Design doc: https://docs.google.com/a/google.com/document/d/1s0yAfMu53KAqCkrONvcbSwoivnHSdp6ybHk0ksK_QfY/edit?usp=sharing

This is replaced by #6414, #6373, #6438


Page breaks on page change

Describe the bug
The page content is not updating when clicking a link or using the goto('/path') method in Svelte 5. After clicking a link, the old page content is removed, but the new page content is not inserted into the DOM. No errors or warnings are logged in the console.
Expected Behavior:
The new page content should be inserted into the DOM when navigating to a new path using goto('/path').
Reproduction
stackblitz
github
Run the development server.
Click on a link.
Observe that the old page content is removed.
Note that the new page content does not appear in the DOM.
Logs
No response
System Info
System:
    OS: macOS 14.5
    CPU: (8) arm64 Apple M1
    Memory: 81.42 MB / 8.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.9.0 - ~/.nvm/versions/node/v20.9.0/bin/node
    Yarn: 1.22.19 - ~/.nvm/versions/node/v20.9.0/bin/yarn
    npm: 10.2.4 - ~/.nvm/versions/node/v20.9.0/bin/npm
    pnpm: 9.4.0 - ~/.nvm/versions/node/v20.9.0/bin/pnpm
    bun: 1.1.4 - ~/.bun/bin/bun
  Browsers:
    Chrome: 126.0.6478.127
    Safari: 17.5
  npmPackages:
    @sveltejs/adapter-auto: ^3.0.0 => 3.2.2
    @sveltejs/kit: ^2.0.0 => 2.5.18
    @sveltejs/vite-plugin-svelte: ^3.0.0 => 3.1.1
    svelte: ^5.0.0-next.1 => 5.0.0-next.167
    vite: ^5.0.3 => 5.3.2

Severity
blocking all usage of SvelteKit
Additional Information
This issue is only present in the development server.
It is not reproducible in Svelte 4; it works as expected.

Closing as duplicate of https://github.com/sveltejs/svelte/issues/12235


Fix silence detection

PR for the issue https://github.com/argmaxinc/WhisperKit/issues/27.

Added SilenceDetectionFilter similar to LanguageDetectionFilter.
Implemented detectSilence similar to detectLanguage in TextDecoder that does a single forward pass from  to detect silence periods.
Added tests for SilenceDetectionFilter.
Added silence detection and segment skipping logic in TranscribeTask.
Updated models to support silence detection:

Added flags in DecodingOptions for ignoring prefill prompts during silence detection.
Changed property for managing silence detection threshold.

Hey @aigerimmmm just checking in, how are you feeling about the changes?

@aigerimmmm Checking in, is this ready for another review?

Hi @ZachNagengast thank you for reply, yes I am ready anytime for another review.


Terminal Apps No Transparency

Hello,
I've been trying to make my terminal applications automatically open with transparency applied, However they require running the wal -R command while they're open for it to apply. Is there any way I can fix this? I have attached images.
Apps are being opened with i3 shortcuts, " terminal -e 'ranger' "
How it opens

How i want it

Fixed the issue. Required using Xdefaults and copying a few settings over from the cached colors.Xresources file


feat: add non-mentioning links option

Pull Request
Adds a new env var which allows you to specify non-mentioning links to the issues.
Proposed Changes
The links still work as before, but they do not cause a notification in the desitnation repository, which helps reduce "spam" there.
Readiness Checklist
Author/Contributor

[x] If documentation is needed for this change, has that been included in this pull request
[x] run make lint and fix any issues that you have introduced
[x] run make test and ensure you have test coverage for the lines you are introducing
[x] If publishing new data to the public (scorecards, security scan results, code quality results, live dashboards, etc.), please request review from @jeffrey-luszcz

Reviewer

[ ] Label as either fix, documentation, enhancement, infrastructure, maintenance, or breaking

Kindly requesting review from @jeffrey-luszcz as instructed :)


gocron crashes while scheduling large number of events

to reproduce add following to main in example.go
	for ind := 0; ind < 10000; ind++ {
		gocron.Every(1).Minute().Do(task)
	}

This fails with error:
panic: runtime error: index out of range

goroutine 1 [running]:
main.main()
	/GOPATH/goscheduler/examples/example.go:24 +0xbbe

I have tried slowing the for loop by adding time.Sleep(1*time.Second), still it fails.
PS: Is this package under active development?

Had to edit MAXJOBNUM from 10,000 to something greater to fix this.


add info on performance_report

Doc update on using the performance_report context manager

Done!  Thank @jrbourbeau for the review

Thanks!


How To Bulk Edit - selecting the thing to replace

We saw a demo of Views Bulk Edit today in the sandbox call.
Something we may want to do is to replace specific values with a different value.  once we've used our bulk-edit view to find and select the items with this specific field value, we are able to:

Append a new field value to the objects
Replace all of the existing values in the field with the ones we enter in the bulk edit interface.

What it does not allow is, for multi-value fields, replacing only the target value with a different one.

that's dangerous

Yes! But it is part of the process of "we migrated before we had cleaned up our data", and is also part of the work of having authorities. Is there an export-import process that would work for this?

personally i'd probably do an export as CSV, clean up the data, and do a migration to update the records.

I'd be curious to know if any metadata-focused system can do this using its standard metadata editing tools.
I would recommend @elizoller 's approach. I assume that Migrate lets you indicate that you're overwriting all values in a field with the incoming ones, but Islandora Workbench has an open issue about providing an "append", "delete" (all values in field), and "replace" update action, which I've been putting off to come after a major refactoring of the code that deals with different Drupal field types.


Move to a custom matcher for SQL result checking

Not 100% sure this is how we should go, but after working in a couple of different problems, I have managed to abstract out a custom matcher which I find useful.
It has the following features

The code to test a malloy fragment against an expected value result is pretty short and clear

      test(`timestamp delta hours - ${databaseName}`, async () => {
        const result = await sqlEq(
          "t_timestamp + 10 hours",
          "@2021-02-24 13:05:06"
        );
        expect(result).isSqlEq();
      });

When it fails produces a dump which shows you the expected value, what it got back, and the SQL that was used to compute the result.

  ● time operations - postgres › delta - postgres › timestamp delta hours - postgres

   t_timestamp + 10 hours != @2021-02-24 13:05:06. Got: 2021-02-24 13:00:00-10
   SQL:
   WITH __stage0 AS (
     SELECT
        DATE_TRUNC('second', TIMESTAMP '2021-02-24 13:05:06') as "expect",
        DATE_TRUNC('hour', ((basicTypes."t_timestamp")+make_interval(hours=>10))) as "got"
     FROM (
         SELECT
           DATE('2021-02-24') as t_date,
           '2021-02-24 03:05:06'::timestamp with time zone as t_timestamp
        ) as basicTypes
   )
   , __stage1 AS (
     SELECT
        CASE WHEN DATE_TRUNC('second', base."expect")=DATE_TRUNC('hour', base."got") THEN '=' ELSE concat('t_timestamp + 10 hours != @2021-02-24 13:05:06. Got: ',cast(DATE_TRUNC('hour', base."got") as varchar)) END as "calc"
     FROM "__stage0" as base
   )
   SELECT row_to_json(finalStage) as row FROM __stage1 AS finalStage

How I accomplish this is a little wonky and needs review though.

@bporterfield I've made a custom matcher private to malloy-db-test/src but I don't know if the machinations I had to go through to get there are something we can support or if they just accidentally work.
@lloydtabb this is how I think we should write tests, if you like it and ben approves, I'll move all result tests to this pattern.


bootable key does not start

I followed the guide perfectly but unfortunately when i insert the bootable key in my t440p it doesn't recognize opencore. I have set up my bios like guide and my bios is updated to the latest version of lenovo. Where am I wrong?

Same issue here.

Hey, sorry for the late reply. 😅
You need to press SPACE to see the installer / tools.
I will update the README asap.


MQTT mode isn't easy to share with multiple requestors

With the structure given in the readme:
request_topic: doods2/request_topic
response_topic: doods2/response_topic
This means multiple clients making requests would all get the responses on the same topic.
I would recommend a simple enhancement:
request_topic: doods2/request_topic/+
response_topic: doods2/response_topic/+
Where the last part of the path would be a unique client ID that the client makes up and the response message would have the same ID at the end.  That way multiple clients can make up their own client ids and subscribe to them and then not get responses intended for other clients.

There is an ID field in the request topic. You can put whatever you want in there. The same ID will be returned in the response. You can use this to multiplex message. As well, there's not a real easy way to monitor zillions of request/response topics. You can change the config file to monitor a different request/response topic if you like and use multiple instances of DOODS.

@snowzach  That's my whole point, DOODS2 could trivially subscribe to doods2/request_topic/+ as a single subscription and catch requests from every requestor, but the client could post that request with their process id (1234) and then subscribe to doods2/response_topic/1234.  This completely removes the need for every requestor to subscribe to EVERY response sent to every requestor and then throw away every response that doesn't have their ID in the message.  That client can keep using that process id and response subscription and only get responses meant for them for the life of the client process.
I've used this technique for servers I've written and it's far more efficient and very easy to implement.

I'll look at it whenever I get a chance.. Happy to take a PR as well..


Add link to UML diagram

Let's try to keep the UML up-to-date...

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@3ffcdcf). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #92   +/-   ##
=========================================
  Coverage          ?   87.25%
=========================================
  Files             ?       27
  Lines             ?     1765
  Branches          ?        0
=========================================
  Hits              ?     1540
  Misses            ?      225
  Partials          ?        0

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 3ffcdcf...07166fc. Read the comment docs.


Feature Request: Non-sampled API

I'm toying around with what an impl for this might look like but wanted feedback before spending much time on it.
For the most part I'm not super worried in being able to configure the sample rate for each individual stat that I'm collecting -- there are likely to be some things on the hot path that I'll want to turn down and some low-volume-but-essential things I need 100% visibility. It would be nice if there was a way to build a Statter with preset sample rates so that when you call Inc etc you don't need to provide your own sample rate.
Expressing this as code (pretending we get method overloading):
func (s *Client) Inc(stat string, value int64, rate float32) error { ... }

func (s *Client) Inc(stat string, value int64) error {
    return s.Inc(stat, value, s.sampleRate)
}

Is this something you would not want merged in or is it worth poking at a bit more?

I think I would be hesitant to add that to the library, based on my current understanding of your intent. This seems like something that would be a better fit for either application specific code, or possibly a higher level convenience wrapper.
I certainly think it is worth fleshing out further though. There are likely many other conveniences/niceties that would be a better fit in a convenience wrapper layer, built atop go-statsd-client, that may not make sense at the level of abstraction targeted here.

After spending some time trying to shoe horn this as an expansion of the core feature set without reaching modifications I ended up at the same place. It works reasonably well as a layer on top (there is some wonkiness because it breaks the assumption that only a SubStatter will include sampling.
Anyway, it'd look something like https://github.com/falun/go-statsd-client/commit/379bc770813199c37fa6cbf302f951e73d45046f -- would you want that contributed back as beginning of a collection usability-enhancing wrappers?

I think for now it makes sense to have these types of wrappers stay external. Closing for now.


iPad - UI Inconsistency of the popover menus - reported by  @Santhosh-Sellavel

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

Action Performed:

Open app
Navigate to "+"
Select any option

Expected Result:
Upon selection, full button should be highlighted. Either options should look like as it is desktop or buttons should be clickable for full width.
Actual Result:
Button is not fully highlighted
Additional areas where issue is occurring

Compose Attachment options
Call Options
Message Context Options
Workspace ( Three dots) More Menu Options
Add Payment Options
Add/Edit Photo Options for Workspace & User Profile.
Transfer Balance Options

Workaround:
unknown
Platform:

Where is this issue occurring?

iOS

Version Number: 1.1.32-0
Reproducible in staging?: Y
Reproducible in production?: Y
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

https://user-images.githubusercontent.com/43995119/150906959-44d501c9-3428-494f-bc9a-8f11d602c5e8.mp4
Expensify/Expensify Issue URL:
Issue reported by: @Santhosh-Sellavel
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1641315894001600
View all open jobs on GitHub

Triggered auto assignment to @Gonals (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

Proposal
Add prop fromSidebarMediumScreen={!this.props.isSmallScreenWidth} to all Popover components, wherever the issue was noticed.
E.g. ReportActionCompose (line 564)

This issue has not been updated in over 14 days.  eroding to Weekly issue.

This issue has not been updated in over 14 days.  eroding to Weekly issue.

This issue has not been updated in over 14 days.  eroding to Weekly issue.

This issue has not been updated in over 14 days.  eroding to Weekly issue.

This issue has not been updated in over 14 days.  eroding to Weekly issue.

This issue has not been updated in over 14 days.  eroding to Weekly issue.

This issue has not been updated in over 14 days.  eroding to Weekly issue.

 @Gonals did you want this pushed externally?
If so I'll assign myself.

Ah damn! Yeah, I think this is a good candidate for external

Assigning myself as External.

Posted on Upwork: https://www.upwork.com/jobs/~0112ee5bb8a9f27309

Triggered auto assignment to Contributor-plus team member for initial proposal review - @parasharrajat (Exported)

Triggered auto assignment to @Julesssss (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

@srdjanrist I don't understand the purpose of your proposed change. Could you please explain what will this prop do?

Hello,
Not seeing code it is hard to see a problem and fix it but I do have a couple of possible solutions. Try setting the button width to Dimensions.get('window').width. Another solution is creating a pressable with a view component inside. Make the view component the size you want. Then when you click the pressable change the view's background color.  I would love to help but without code it's quite hard.

Hello @jkgaming88 ,
It is OSS and you can head over to Read.me for more..

Hello @parasharrajat
Fixed the problem. Change width in popoverMenuItem style to Dimensions.get('window').width. The '100%' was causing the issue. This makes the menu item the size of the modal. It will never exceed it. You could set the value to 10,000 and it will still work. I think React Native is thinking 100% width is the size of the initial pop over. This fixes that
https://user-images.githubusercontent.com/79109856/155439735-c7c61a18-7e33-4c89-b8ae-347cd3688c3f.mp4

This is the initial popover I meantioned before. Notice how the width of the popover is the same as the button pre-fix. I do not know why it happened but that is the reason.

@srdjanrist I don't understand the purpose of your proposed change. Could you please explain what will this prop do?

Purpose of fromSidebarMediumScreen is to decide which type of modal will be shown.
If fromSidebarMediumScreen = true POPOVER modal will be shown.
If fromSidebarMediumScreen = false DOCKED modal will be shown.
As we can see on iPad 11 inch, in the side menu, current behavior is to show popover, instead of docked modal. (behavior is similar to what can be seen on web).
Also, on the same device, we can see that the screen is split, meaning it shows sidemenu with other reports, and on the right, current report is shown. (again behavior similar to web).
By setting the flag fromSidebarMediumScreen to what I have proposed, we will have consistent behavior, where it will always show popover, instead of docked modal.
This can be seen on the following images.
Current behavior:

Proposed fixed behavior:

Ok Sounds good @srdjanrist . I like @srdjanrist 's proposal.
cc: @Julesssss
:ribbon: :eyes: :ribbon:  C+ reviewed

fyi @Julesssss let me know if you're happy to proceed with @srdjanrist 's proposal!

Thanks all, yep the proposal sounds good.

📣 @srdjanrist You have been assigned to this job by @Julesssss!
Please apply to this job in Upwork and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑💻
Keep in mind: Code of Conduct | Contributing 📖

Perfect, thanks @Julesssss .
@srdjanrist I've just hired you in Upwork!

Not overdue Melvin, we have a PR that's being worked on.

Reassigning CM to @kadiealexander.

@srdjanrist Could you please provide an update on PR?

Thanks for waiting for me @parasharrajat . I have updated the PR with a few commits. So you can check them.

@srdjanrist I can see the last comment on the PR was 5 days ago, could you please leave a comment here letting us know where you're up to?

@srdjanrist I can see the last comment on the PR was 5 days ago, could you please leave a comment here letting us know where you're up to?

Thank you for the heads up. I will check those few things mentioned in the PR today (28 March).

Okay, seems like we're seeking a new contributor here. Open to proposals again!

Yup. Review would be done today.

Bumping this. Seems like we're at the review stage?

@srdjanrist could you please provide an update for us here?

Looks like the PR is still moving along

Awaiting changes based on feedback, I bumped one of the unresolved threads

Won’t be able to continue working on the proposed changes. Someone can take over. Thank you
Get Outlook for iOShttps://aka.ms/o0ukef

From: Jules Rosser @.>
Sent: Tuesday, May 3, 2022 11:25:16 AM
To: Expensify/App @.>
Cc: srdjanrist @.>; Mention @.>
Subject: Re: [Expensify/App] iPad - UI Inconsistency of the popover menus - reported by @Santhosh-Sellavel (Issue #7375)
Awaiting changes based on feedback, I bumped one of the unresolved threads
—
Reply to this email directly, view it on GitHubhttps://github.com/Expensify/App/issues/7375#issuecomment-1115897968, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGBXC4RLLM4WDFBDIG67PULVIDWHZANCNFSM5MXEGXYA.
You are receiving this because you were mentioned.Message ID: @.***>

Hi @srdjanrist, sorry to hear that. Is it because of the amount of time/effort taken? Because if so, the job prices can always be raised to match the effort.

Hey @Julesssss thank you for the response.
It has a little bit to do with time involved so far. Main reason is that I can't commit myself for long periods of time and focus on the issue ahead (mainly because I started working on another project, while working on this issue). And due to that, since I am a responsible person, I don't want to drag you along and slow everything down.
Certainly, I would like the code that was already made into a pull request, to be used, so that it doesn't go to waste fully.
I had a really nice plan to work on the issues on Expensify, but because of other obligations those plans didn't really end up well.

Taking over the PR

Not overdue Melvin.

Still on my todo list

Please get your proposals in! This one is half-finished so it's an easy one to pick up!
https://www.upwork.com/ab/applicants/1532271886227472384/job-details
https://www.upwork.com/jobs/~01b7fcbbd791dc6c42

I was looking into this. And I tried a few ways to get the correct position. But In all of them I didn't get the correct coordinates. I don't know why but none of the possible ways are giving correct coordinates for PaymentMethod of PaymentMethodList on  Android Tablet.
I used onLayout, measure, measureInWindow, PressEvent.

Oh, I see now.measureInWindow could be wrong due to SafeArea insets. Need to check that more.

@parasharrajat any further thoughts on this? Otherwise, I'll double the price again :)

@kadiealexander No new thoughts yet.

Thanks Rajat! Doubled the price team, get your proposals in!

Proposal
Instead of trying to show popover menu as side mounted on native platform, we can keep it as it is and fix the the problem with menu item width.
Solution
The following changes:
https://github.dev/Expensify/App/blob/5f7ecce02299e9eca05112d8de1270063cb45dd0/src/components/AttachmentPicker/index.native.js#L311
to:
<View>

https://github.dev/Expensify/App/blob/d6a607324f84e654da2da6e073b150a00def7490/src/components/PopoverMenu/BasePopoverMenu.js#L41
to:
<View style={!this.props.fromSidebarMediumScreen ? {} : styles.createMenuContainer}>

This should fix most of the popovers on native. However, we need to follow a similar approach with the remaining popovers (ConfirmPopover and PasswordPopover). We should create native only files for the above mentioned popovers and ignore any width property being set on the content.
Result:
https://user-images.githubusercontent.com/30054992/174228623-e9768ed1-ac0c-4997-9926-80416d31a39f.mp4

@allroundexperts We want to change the menu position as discussed above.

So fixing bottom mount is not an option @parasharrajat ?

I spent some time yesterday to look into the issue more. As said, problem is https://github.com/Expensify/App/issues/7375#issuecomment-1152367769.
So I have found a solution to it and I can complete the Old PR. I have it ready let me know if you want to push it.
cc: @kadiealexander @Julesssss.

🙅

So fixing the bottom mount is not an option @parasharrajat ?

The full button UI does not look good. Old. So a better solution would be to move it to the side in place.

@parasharrajat, I think the blocking issue is getting solved as a different issue https://github.com/Expensify/App/issues/9283

That one is handling web. This one will fix native.

@Julesssss What do you think? Should we keep the menu at the bottom or move it to the side-window as a desktop?

I agree the full width does not look good. We should have updated the expected result too.
Please go ahead and push your fix!

Thanks, I will do that asap.

PR is created. But there is some delay before the menu is shown as coordinate calculations are done asynchronously. I will look into alternatives to remove the delay. I have already tried three different implementations but none of them removed the delay/ was most effective.

Pass ref and calculate position via ref.measure. Current PR.
Use onLayout to get coordinates and pass them instead of ref. It does not work with react-navigation. onLayout does not fire in sync with animations. So there is no guarantee that onLayout will give you the correct coordinates.
Use ref callback and calculate position via ref.measure inside ref callback. It is bad performance-wise.

Not overdue Melvin, @parasharrajat is working on this PR.

Missed it. I will update the PR soon.

Testing the PR after merging main and then upload the videos.

There are two other PRs that are trying to fix different aspects of this page. I don't want to make things harder for them so I will wait for them to be merged and then refactor that code.
https://github.com/Expensify/App/pull/9622
https://github.com/Expensify/App/pull/9552

On hold! Shifting to monthly while we wait.

PRs still not merged, still on hold.

@parasharrajat how are we tracking with this one?

Oops, was this monthly, I didn't get a notification. Looks like both PRs are merged. I will analyze the current state and update the PR accordingly asap.

Bumped to Daily and removed hold in title to get things moving again

Hey @parasharrajat , now that the other PRs are merged I think we just need to retest and confirm that it's occurring, right?

Yes, https://github.com/Expensify/App/issues/7375#issuecomment-1284581220. I will update it as soon as I get time to work on it.

Not overdue

Busy with a very high-priority issue.

Hey @parasharrajat, is now a good time? Or maybe we could reassign if you're still busy?

Let me check this. I will either update the PR by tomorrow or let you know if reassign is needed.

Analysing the latest code and my solution now.

Ok, I tested. The problem remains the same for dynamic menus.
Problem:

For dynamic content(whose position is not fixed), we need to use measureInWindow to get its position which is async. Due to async nature, it create a small delay before menu can be shown.

I faced this issue with the menu of the added payment method (the list). I tried to get rid of this delay but had no luck.

This BasePaymentsPage use getClickedElementLocation to get the position so I did https://github.com/parasharrajat/Expensify.cash/blob/popover-fix-rajat/src/libs/getClickedElementLocation/index.native.js
@Julesssss Do you have some suggestions for it?

Thanks for the detailed investigation. That's unfortunate, and I don't have any suggestions for the async workaround.
Perhaps we should double the budget and seek additional proposals

Ok, let's open it back to contributors. BTW,I am C+ on this issue. I was trying to complete the partial PR.

@kadiealexander Please add the Help Wanted label and open the Upwork job(I think the old one got closed) once you have time. Thanks.

Please get your proposals in!
https://www.upwork.com/ab/applicants/1587903501951631360/job-details
https://www.upwork.com/jobs/~011dc35409294bd219

@kadiealexander reminder these should remain daily now!

Price doubled!

Proposal
The issue is because the parent component of the Menu Items have a fixed width for non "isSmallScreenWidth".
The solution consists in giving to that parent component a style prop "width" with a 100% value and also another small change to cover the Desktop case.
I already have a working solution that only changes 2 lines.

Proposal
The issue is because the parent component of the Menu Items have a fixed width for non "isSmallScreenWidth" gigen by the "createMenuContainer" style.
The solution consists in giving a full width to the parent of the menu items in the case of a native device. Also taking into account the special case for the SidebarMenu in native menu.
I already have a working solution that only has small changes of styles.

Oh Shoot, the expected behavior is wrong.
https://user-images.githubusercontent.com/24370807/201417380-ca5a0d73-2ff6-4804-b382-9b245f4c1c9e.mp4
A long time ago, we decided that the native app on tablet or ipad should show menus like on web | desktop. No Full width menus.
@Julesssss Can you please confirm this?

@parasharrajat hmm, yes that is a great point. So I believe we would expect the following:

Yes, that would be the ideal solution!

Hi. How do you usually proceed with this?. You develop it internally? Create a new ticket? This issue is still valid?

The issue is still valid and open for proposals. @Julesssss Can you please update the Expected result section?

Proposal
Some changes need to be made in some components: Popover.native to set as Popover in medium devices (Ipad/tablets), And calculate the popover position in AttachmentPicker, ReportActionCompose and AddPaymentMethodMenu.
Some areas such as Call Options and Add/Edit Photo Options for User Profile don't need any code changes. Will check other areas if my proposal is accepted.
Video of my working branch
https://user-images.githubusercontent.com/2470879/201991522-b0550ad0-7b0a-4a47-a650-7eb4d18d78b6.mp4

If you follow the step above you should be good with it. Notice I mentioned adding the account from the web app as Native currently uses PROD API. Plaid flow is pretty easy just keep continue to next steps.

Hi @parasharrajat. Any idea why the "getClickedTargetLocation" method would not work properly on Native events? Any related issue apart from https://github.com/Expensify/App/issues?q=getClickedTargetLocation ?

thanks

It is not implemented on native https://github.com/Expensify/App/blob/main/src/libs/getClickedTargetLocation/index.native.js

No idea, I am expecting a solution via proposals.

How will you reduce the delay that is taken for the menu to open?
There is a small flicker when we open attachment menu on composer, how will you solve it?

Hi @parasharrajat.
About your questions:

How will you reduce the delay that is taken for the menu to open?

I don't see what are you referring to. Can you say to me in which exact second of the video is happening?

There is a small flicker when we open attachment menu on composer, how will you solve it?

Fixed. It was the position being used from an old state when opening different popovers on that page. To reproduce it we had to press the "Add Payment" button and then "Existent Payment" button.

Updated video
https://user-images.githubusercontent.com/2470879/202235862-3834ee90-3eae-4fb9-9645-6d2b7dbbc201.mp4
Many thanks

Can you say to me in which exact second of the video is happening?

0.27 on the first video. It takes a bit of time to open as the measure takes time to resolve and it is async.

Hi. The button works fine with the last changes.
Proposal

Change the modal type from "BOTTOM_DOCKER" to "POPOVER" for medium/large native devices
Implement "getClickedTargetLocation" method for native devices
Position the "Message Context Options" popup
Position the "Add attachment" popup
Position the "Add Payment Options" popup
Position the "Avatar photo " popup for Workspace & User Profile
Position the "Transfer Balance Options" popup
No changes in "Call Options" popup
No changes in "Workspace ( Three dots) More Menu Options" popup

Many thanks

It would help us if you can explain the changes you would do to the app for this issue.
Links:

CONTRIBUTING Guidelines
App overview and local development
CODE OF CONDUCT
Code style and patterns
Styling Components

I would recommend checking out some previous jobs, which already got a contributor assigned to see how they posted the proposals. There is also this part in our contributor guidelines which you might find useful.
Please, also share the exact changes in code you had to make to fix this issue. You can see from all the other issues and contributors that we won't just take the code and run away, no need to worry about that :)
Thank you!

Hi. Same steps as the last message but with code.
Proposal

Change the modal type from "BOTTOM_DOCKER" to "POPOVER" for medium/large native devices

Implement "getClickedTargetLocation" method for native devices

Position the "Message Context Options" popup

Position the "Add attachment" popup

Position the "Add Payment Options" popup

Position the "Avatar photo " popup for Workspace & User Profile

Position the "Transfer Balance Options" popup

Many thanks

The code is looking good. As I worked on this task earlier, there were a few challenges.
We are using measureInWindow in Implement "getClickedTargetLocation" method for native devices. It delays the menu opening after you click the button. You can also see this in your videos. How will you solve this?
this is one of the major challenges I stuck at earlier.

Hi @parasharrajat. I don't came up with an alternative to measureInWindow.
I suggest to calculate the Popup positions on the background once the page is loaded. This would not affect the page time load and the position will be ready before he user try to press the buttons.

I suggest to calculate the Popup positions on the background once the page is loaded.

Ok. What about the menus of dynamic anchors? Let's say the anchor move position. This can be observed on the payment method list. I onLayout does not work well. It does not fire all the time in Animatable Views.

@Julesssss To help clear out the extensive backlog of /App bugs, we're putting the spotlight on all bugs older than 4 weeks old. To help unblock the roadmap and get our bug pipeline back in equilibrium, can you:

Decide whether any proposals currently meet our guidelines and can be approved as-is
For any that can't, please take this issue internal and treat it as one of your highest priorities
If you have any questions, don't hesitate to start a discussion in #bug-zero

Thanks everyone!

Based on our discussion in the community, I'm going to close this issue. Given that tablets are not an officially supported platform, this issue is more accurately a new feature request than a bug. Closing.

Hi @parasharrajat. Can you update to me the status of the issue?. You said to me this issue was still valid and It wasn't to be developed internally.
Many thanks.


Website

[x] setup sass
[x] choose fonts
[x] find favicon
[x] add code highlighter
[x] add nav links to github, patreon, slack
[x] style footer
[ ] add social media icons
[ ] add links to clients

https://github.com/io-ma/fist-site

We are life at fist-project.com

I have to polish it a bit, will do it asap

Oh true, I will add HTTPS. Mobile does require a bit of work but otherwise everything is great. I just need to edit the content a bit. I also moved Client Libraries to its own section, I don't know if you saw.

Just noticed now. Makes sense.
How about I link it in the nav, with a Libraries ?
It's more accessible.


Make sure records in parenthesis are treated the same

In code where sub-records were defined in a parenthesis, we weren't pushing the parent's identity down into the children, we also weren't setting eve-auto-index on them. This makes sure we do both and should remove any differences between using parens or not.
Fixes. #673

eve-auto-index serves a pretty important pragmatic purpose, but maybe there's some other way to accomplish what it gives us. The main value is in something like this:
bind @browser
  [#div children:
     [#div text: "foo"]
     [#div text: "bar"]
     [#div text: "baz"]]

Since eve-auto-index is applied to these, we know to keep them in that order on display. If we didn't have something like that you'd need to supply sort on all of them despite the fact that we've already expressed the intent.

Where that overhead would really get you is if you were inserting something in the middle and then had to update every sort attribute underneath. Do that a couple times and it'll seem worse than busy work.


Correct warden URL in README [ci skip]

Warden moved to its own organization so we should follow it.

Thanks!


Parser fails occasionally

From your bass repo:
examples/bass/bass/bass.bass                                                    1 ms    (ERROR [45, 34] - [45, 35])
examples/bass/demos/go-build-git.bass                                           0 ms    (MISSING symbol [9, 6] - [9, 6])
examples/bass/demos/multi-fail.bass                                             0 ms    (MISSING symbol [5, 6] - [5, 6])
examples/bass/pkg/runtimes/testdata/lib/thunk-path-lib.bass                     0 ms    (MISSING symbol [5, 17] - [5, 17])
examples/bass/pkg/runtimes/testdata/mount-run-dir.bass                          0 ms    (MISSING symbol [8, 48] - [8, 48])
examples/bass/pkg/runtimes/testdata/remount-workdir-subdir.bass                 0 ms    (MISSING symbol [12, 28] - [12, 28])
examples/bass/pkg/runtimes/testdata/remount-workdir.bass                        0 ms    (MISSING symbol [5, 17] - [5, 17])
examples/bass/std/git.bass                                                      0 ms    (MISSING symbol [23, 27] - [23, 27])
examples/bass/std/run.bass                                                      0 ms    (ERROR [59, 44] - [59, 46])

I'll work on this a bit, but I'm not too familiar with bass. My plan was to tidy up the repo and then add it to nvim-treesitter.

Interesting, thanks for the report!
It looks like all of these failures are related to path parsing, which I had the most difficulty implementing by far. :sweat_smile:
Some pointers:

paths
thunk paths

The tl;dr is that you can do things like this:
(def my-path  ./)
(def sub-dir  my-path/subdir/) ; <- trailing slash
(def sub-file my-path/subfile) ; <- no trailing slash

This is a little tricky parser-wise because it means any symbol can turn into a path by affixing /foo[/] to it.

I'll work on this a bit, but I'm not too familiar with bass. My plan was to tidy up the repo and then add it to nvim-treesitter.

That's exciting! Bass is a bit of a fringe language, so I'm not sure if it would be accepted upstream, but I'd certainly appreciate any time you find to help fix this parser up.

nvim-treesitter has some of the wackiest languages added, don't worry bass would be totally accepted. I am kinda thinking to rewrite parts of this (I've written quite a few grammars) to alleviate this issue, but I'd like your nod of approval before spending some time on this.

@amaanq By all means, rewrite if you like! This was my first grammar and it's just based on the most similar starting point I could find.


[HUDI-1059] Adding test to WriteClient to validate update partition path with global bloom

What is the purpose of the pull request
Adding a test to validate update partition path
Brief change log
Verify this pull request
this patch is just adding tests
Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@xushiyan Could you please review this PR since you did the related work before.

@nsivabalan this is more than the 50 lines we agreed on for MINOR prefix. can you please file a JIRA and let me know if you think this quallifies for the MINOR prefix.

@xushiyan Could you please review this PR since you did the related work before.

@leesf Yes I can look into this soon.

@xushiyan : can you review this when you get time.

Closing this in favor of https://github.com/apache/hudi/pull/1793


Unable to clone private repository from Github

Dev Home version
<IP_ADDRESS>
Windows build number
10.0.22621.1702
Other software
OS Build Version: 10.0.22621.1702.amd64fre.ni_release.220506-1250
.NET Version: .NET 6.0.16
Steps to reproduce the bug
Followed the setup wizard and did not install any application that it recommends but wanted to clone the repo and open it in VSCode

End-to-end setup and add repository
added github private repository, select the repo from "Account" section and click next
None installed from "Install Applications" section
Agree to proceed the setup
error

Expected result
Clone repository
Actual result
Unable to clone the repo

Included System Information
CPU: Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
Physical Memory: 23.91GB (15.16GB free)
Processor Architecture: x64
Included Extensions Information
Extensions:
Microsoft.Windows.DevHome_<IP_ADDRESS>_x64__8wekyb3d8bbwe
Microsoft.Windows.DevHomeGitHubExtension_<IP_ADDRESS>_x64__8wekyb3d8bbwe

is this repo in your personal account, or part of an organization?

Personal account


Running the contributed command: 'gitlens.views.showAllChildren' failed

GitLens Version: v10.2.2
VSCode Version: 1.47.0-insider
OS Version: Windows_NT x64 10.0.18362

message:
Error running command gitlens.views.showAllChildren: Running the contributed command: 'gitlens.views.showAllChildren' failed.. This is likely caused by the extension that contributes gitlens.views.showAllChildren.

same. getting below error after today's insiders update. expanding commits/branches/remotes not working
Error running command gitlens.views.refreshNode: Running the contributed command: 'gitlens.views.refreshNode' failed.. This is likely caused by the extension that contributes gitlens.views.refreshNode.

For me, none of the lists in the GitLens extension open up. This is the error in the VSCode DevTools that I saw when I clicked "Refresh" in GitLens:

same. getting below error after today's insiders update. expanding commits/branches/remotes not working
Error running command gitlens.views.refreshNode: Running the contributed command: 'gitlens.views.refreshNode' failed.. This is likely caused by the extension that contributes gitlens.views.refreshNode.

Problem no longer happening in latest Insider build:
Version: 1.47.0-insider
Commit:<PHONE_NUMBER>0e48be84d86f5763e126c7c5bf22e0
Date: 2020-06-30T09:01:06.154Z

same. getting below error after today's insiders update. expanding commits/branches/remotes not working
Error running command gitlens.views.refreshNode: Running the contributed command: 'gitlens.views.refreshNode' failed.. This is likely caused by the extension that contributes gitlens.views.refreshNode.

Problem no longer happening in latest Insider build:
Version: 1.47.0-insider
Commit:<PHONE_NUMBER>0e48be84d86f5763e126c7c5bf22e0
Date: 2020-06-30T09:01:06.154Z

yes


Runtime dynamic size array/list/vector of arbitrary type in callable/kernel

Is your feature request related to a problem? Please describe.
I want to record all the vertex in the path. The vertex is a struct which consists of all DSL variables. However I didn't find a good way to record that in the callable as it seems like the size and the index should be determined at "compile time". Is there any suggested way of doing that?
Describe the solution you'd like
Array<Vertex> vertices(n);
...
$for(i, max_depth) {
    auto v = vertices[i];
    v.position = make_float3(1.0f);
}
...

Describe alternatives you've considered
I have tried linked list by using a pointer in Vertex, but this does not seem right as DSL have not actually run the code. Any suggestion on how to properly implement that will also be helpful.

Hi, @MaxwellFang
You may use Local<T> in <luisa/dsl/local.h> which can be constructed with runtime-specified lengths. The element type T can be any DSL type, i.e., scalars, vectors, matrices, or POD structures reflected by LUISA_STRUCT.
Example:
namespace some {
struct Test {
  float a;
  uint2 b;
};
} // namespace some

// reflect the plain C structure
LUISA_STRUCT(some::Test, a, b) {};

int main() {
  uint n = some_runtime_value();
  Kernel1D kernel = [] {
    Local<Test> x{n}; // construct an array with n elements of struct Test
  };
}

My struct is something like
struct Vertex {
    Float3 position;
    SampledSpectrum beta;
    ....
};

which the variables DSL types but the struct itself is not reflected by LUISA_STRUCT. So I was wondering if that would be the same.


[oracle] Update miss calculation for abstaining and unment quorum

This updates the oracle logic to differ from the Terra behavior. We want the oracle to penalize abstaining from votes by counting them as misses, and to also count unmet quorum as miss for validator/oracles that don't provide in-range values to prevent collusion to avoid meeting quorum.

There are a couple TODOs, which pertain to a measure of staleness, which will indicate whether an exchange rate is a stale rate or new. I plan on addressing this in a next PR.


x86_64 iOS template crashes in simulator

Godot version:

3.0
OS/device including version:

Issue description:

exported iOS project can not test on simulator.
Steps to reproduce:
Minimal reproduction project:

All that needs to be done here is to include x86_64 as a platform into export template build scripts.

I mean the lib compiled by the following command crash when run on the simulator
scons p=iphone tools=no  arch=x86_64 target=debug

All that needs to be done here is to include x86_64 as a platform into export template build scripts.

Actually there is x86_64 support in the platform/iphone/detect.py already, so it should work. But the resulting binary seems non functional according to @alexzheng.

Any exported project just crashed here when running in the simulator.
int OSIPhone::get_unused_joy_id() {
return input->get_unused_joy_id();
};

Ah, right, I fixed this locally few days ago.

Will it be possible to make the default static lib file in the template project support both the devices and simulator. It would make test much easier without compiling custom lib.

Same as here: #17331


[Bug]: Service counter is not displaying

Expected Behavior
I'm trying to use serviceCounterLabel to display the tiny counter indicating how many services are in a category. The categories are defined like this (simplified example):
categories: {
    necessary: {
        enabled: true,
        readOnly: true,
    },
    media: {
        services: {
            youtube: {
                label: 'YouTube',
            },
            vimeo: {
                label: 'Vimeo',
            },
        },
    },
}

The translation defines a serviceCounterLabel (under translations.de.preferencesModal):
serviceCounterLabel: 'Dienst|Dienste',

I also tried a simple string without the separator (Dienst(e)), but it's not working. The counter just doesn't show up. Is there something else I need to configure to get the counter to show up?
Current Behavior
The counter icon never shows up, even if included in the config.
Steps to reproduce
Include the serviceCounterLabel in the config as described above.
Proposed fix or additional info.
No response
Version
3.0.0-rc.15
On which browser do you see the issue?
Firefox

This works fine on my end: demo
Perhaps its just hidden; the label is hidden on mobile view and when the selected layout is 'bar', as the modal becomes quite narrow.

@orestbida You're right, thanks! I was using bar layout. I think this caveat should be mentioned in the docs, I have opened a PR for this: #535


Major change: Update SMO, update dbatools library, and switch System.Data to Microsoft.Data

This is a big one! Since there's so much potential for conflict, I'd like to get this merged as quickly as possible. If there are mild style issues, we can address in a new PR.
When/if you test this branch, remember that you will have to close VS Code and powershell/PowerShell ISE before switching branches as it's replacing a ton of DLLs, both SMO and dbatools.
I've tested this on Linux, Windows 10, Windows 2008 R2 and whatever appveyor is (2012 I think?), all with a surprising amount of luck.

The new SMO failed on tabs in Queries (.Query), so I replaced tabs with spaces in the ps1xml
You may notice some changes to tests -- nothing impactful, I either tried to switch the instance to get appveyor to cooperate or I fixed an issue I've seen for a while
Microsoft.Data generates mildly new connectionstrings that are incompatible with System.Data. Anything that requires a System.Data SqlConnection had to be created using System.Data or the connection string had to be converted
I removed the usage of exclamation points in end-user messages in the dbatools library while I was upgrading the library. i also clarified some error messages. i've wanted to do this for a while and took the opportunity.
The new library was built with Visual Studio 2019 Community Edition though it can be recompiled using the dotnet sdk

Got a couple more tests to fix (Pester syntax errors, mostly) but feel free to review while I clean up

Oh, also, the .NET version is drastically newer than the old one (2 vs 5) but the Desktop version that we target is from 2018. This is reasonable in keeping with trying to accommodate older environments, as 2018 was 3 years ago. My Windows Server 2008 R2 running PS3 that gets updated when it wants to was able to run it, so I'm comfortable with the major update.

Annnnnnnd I rewrote New-DbaConnectionStringBuilder to align with PowerShell standards, as it had a C# flavor. I introduced a Legacy parameter to allow for System.Data or Microsoft.Data and while I was at it, cleaned up. The WhatIf was not useful "making new conn string", so I removed it.

.NET reference for those wanting to know about the 4.6.2 requirement https://docs.microsoft.com/en-us/dotnet/framework/get-started/system-requirements


fix: fix providerName type to allow user to extend default providers

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

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


setuptools.file_finders entrypoint is a giant time sink

Yesterday I fought with getting my build running on a Gitlab CI project. The build failed unhelpfully with this message:
$ strace -o toxfail.trace -f python2.7 /builds/scale-open-source/pyinstaller/setup.py sdist --formats=zip --dist-dir /builds/scale-open-source/pyinstaller/.tox/dist
running sdist
running egg_info
creating PyInstaller.egg-info
writing requirements to PyInstaller.egg-info/requires.txt
writing PyInstaller.egg-info/PKG-INFO
writing top-level names to PyInstaller.egg-info/top_level.txt
writing dependency_links to PyInstaller.egg-info/dependency_links.txt
writing entry points to PyInstaller.egg-info/entry_points.txt
writing manifest file 'PyInstaller.egg-info/SOURCES.txt'
error: [Errno 2] No such file or directory

As you can see from the command I was already resorting to using strace to find out what's going wrong here. In the end it turned out that some freaking file_finders implementation was installed and failed to find the git executable (which is not installed in the container used for the build).
I run into variations of this again and again, most of the time only when checking a package I just created via python setup.py sdist via tardiff against a previous version. Sometimes the size of the package mysteriously doubles - then I can be sure that some dependency pulled in either setuptools_scm or hgdistver or some other package messing with the file lists.
In my opinion this file_finders feature is a bug and must die. Installing a package from PyPI should not cause additional files to be packaged. I'd be fine with a mechanism to explicitly request this from setup.py. Using entry points for this make this just a hidden uber global variable for me.
Workaround
In desperation I added the following command to my CI builds:
python -c 'import pkg_resources;print("\n".join(d.key
        for d in pkg_resources.working_set
        if "setuptools.file_finders" in d.get_entry_map()))' \
    | while read p; do pip uninstall --yes $p; done

And magically all problems seem to disappear.

Ignore that aforementioned commit. I think it was intended for another issue.
You're right - the file finders functionality can be surprising, especially because the behavior is injected by the presence of third-party libs.
I do recommend that these file finder libraries only be installed transiently, as part of a build of a single package. Once PEP 518 lands, it should better guarantee that dependencies like these aren't brought into the build of your package by the build of other packages in the same session.
But as far as deprecating and removing this functionality, I don't think that can happen. It provides too much value and has too many users relying on it. The advantages of the feature outweigh the harm of issues like these.
I don't find that in my environment that these file finders libraries get installed permanently. I suggest you track down what is causing these packages to be installed and see if you can convince them not to install those packages.
Hope that helps.


Failure to parse the Oracle select SQL that contains "not exists"

Steps to reproduce the behavior:
1.String sql = "select * from t001 t where not exists(select * from t002 t1 where t.c1=t1.c1 and t.c2=t1.c2 and ('241' in t1.c3||t1.c4))"
2.Statement statement = CCJSqlParserUtil.parse(sql);
3.Exception:
Caused by: net.sf.jsqlparser.parser.ParseException: Encountered unexpected token: "exists" "EXISTS"
at line 1, column 32.
System
•Oracle
•java 1.8
•JSqlParser version 3.1

The problem is not "not exists" but this in expression without parenthesis. Using in (t1.c3||t1.c4) works fine.

@wumpz  That is true, thanks!

So can we close this? JSqlParser does not (yet) support this kind of construct.

Yes, you can close it, but it is for better to me if can support this construct.

Pull requests are welcome :)


Link Member Withdrawal page

General Checkups

[x] Have you checked that there isn't already an existing issue that describes what you report below?
[x] Have you checked that there isn't already an open pull requests for this issue/update/change?

Description
The member withdrawal page that project leads can use to see which members have left the project and which request data deletion is found at /direct-sharing/projects/withdrawn-members/my-test-on-site-project-name/ but it is not linked anywhere.
The only way to find this page is by receiving an email on withdrawal, but right now getting these emails is optional. The withdrawal page needs to be linked to from the Project management interface on the main site.
This affects both project leads as they never get the information they want and it also affects project members for whom data might not be deleted as project leads never get the message.
Related Issue(s)

Example

Fixed #915


I delete an element from the feed stream

I want to delete a certain element from the infinite feed stream and refresh the display normally. Now the situation is that the last one disappears after deletion, scroll for a while, and the display is normal again. How to solve it  , Here's how
bool removeItem(T item) {
bool result = dataList.remove(item);
if (result == false) return false;
pagingController.itemList = dataList;
notifyData();
return true;
}

Same issue here,
I'm changing the content of the list (add, delete). How to notify those changes?

to correctly notify the controller of the changes, a new list has to be created.
See here: related Stackoverflow issue.

I tried creating uniquekey but the scroll position changes.

Create new PagedListView with around 20 items which can be scrolled down
Add or remove the element
Set new uniquekey
It will scroll to top (Expected - Scroll position should not change)

May be I'm doing something wrong here or I might missed something?


[FIX] SyntaxError: invalid syntax

fix bug: SyntaxError: invalid syntax

This file should not be used. https://github.com/mlc-ai/mlc-llm/pull/322

This file is deprecated, please check MLC-Chat for python usage.

Hi，Do you have a complete use manual to try out this project? I have spend several days to run this project, but got a lot of problems. It really frustrates me. What shoud I do before I use the REST API ? Just run python build.py ???
But now, after I run this REST API, it complains no so file found.
mlc-llm/python/mlc_chat/rest.py", line 35, in lifespan
    raise ValueError(
ValueError: Unable to find vicuna-v1-7b-q3f16_0-cuda.so at dist/vicuna-v1-7b-q3f16_0/vicuna-v1-7b-q3f16_0-cuda.so or dist/prebuilt/lib/v
icuna-v1-7b-q3f16_0-cuda.so.

Then I move the dis folder and file name to the needed one:
vicuna-7b-delta-v1.1-q3f16_0-cuda.so.
It still run failed,
ERROR:    Traceback (most recent call last):
  File "/usr/local/lib/python3.8/dist-packages/starlette/routing.py", line 677, in lifespan
    async with self.lifespan_context(app) as maybe_state:
  File "/usr/lib/python3.8/contextlib.py", line 171, in __aenter__
    return await self.gen.__anext__()
  File "/data/weilong.yu/codes/mlc-llm/python/mlc_chat/rest.py", line 44, in lifespan
    chat_mod.reload(lib=lib, model_path=local_model_path)
  File "/data/weilong.yu/codes/mlc-llm/python/mlc_chat/chat_module.py", line 62, in reload
    self.reload_func(lib, model_path)
  File "/root/codes/tvm/python/tvm/_ffi/_ctypes/packed_func.py", line 238, in __call__
    raise get_last_ffi_error()
tvm._ffi.base.TVMError: Traceback (most recent call last):
  5: TVMFuncCall
  4: mlc::llm::LLMChatModule::GetFunction(tvm::runtime::String const&, tvm::runtime::ObjectPtr<tvm::runtime::Object> const&)::{lambda(tvm::runtime::TVMArgs, tvm::runtime::TVMRetValue*)#1}::operator()(tvm::runtime::TVMArgs, tvm::runtime::TVMRetValue*) const
        at /data/weilong.yu/codes/mlc-llm/cpp/llm_chat.cc:896
  3: mlc::llm::LLMChat::Reload(tvm::runtime::Module, tvm::runtime::String, tvm::runtime::String)
        at /data/weilong.yu/codes/mlc-llm/cpp/llm_chat.cc:253
  2: tvm::runtime::ModuleNode::GetFunction(tvm::runtime::String const&, bool)
  1: tvm::runtime::relax_vm::VirtualMachineImpl::GetFunction(tvm::runtime::String const&, tvm::runtime::ObjectPtr<tvm::runtime::Object> const&)
  0: tvm::runtime::relax_vm::VirtualMachineImpl::GetClosureInternal(tvm::runtime::String const&, bool)
  File "/root/codes/tvm/src/runtime/relax_vm/vm.cc", line 699
TVMError:
---------------------------------------------------------------
An error occurred during the execution of TVM.
For more information, please see: https://tvm.apache.org/docs/errors.html
---------------------------------------------------------------
  Check failed: (finfo.kind == VMFuncInfo::FuncKind::kVMTIRFunc) is false: Cannot support closure with function kind 0

ERROR:    Application startup failed. Exiting.

So Is there a tutorial helping from the beginning to compile a model and then run it?
Thank you!


Fix disk integration tests.

pep8 v1.7.0 was released 6 hours ago. It's creating all kinds of problems. I'm going to kill this PR, submit another PR that specifies an older pep8 version, and then try again. I'll also create a separate issue to update our pep8 version.


Added reasonPhraseExtended()

Added reasonPhraseExtended() to allow retrieval of extended human-readable description text
Added some missing codes

I think, on reflection, my changes can be discarded if you wish. I don't think I need reasonPhraseExtended() at all now. If my other tweaks are useful then please feel free to adopt them, or just reject the pull request.

I've replaced reasonPhraseExtended() and replaced it with reasonPhraseEx() which emits "HTTP : phrase" where a phrase is available or "HTTP " if not.
I have not addressed your other review comments at this time. I have no strong feeling about keeping those changes, as I don't really need them. I'm happy to revert them.

I've replaced reasonPhraseExtended() and replaced it with reasonPhraseEx() which emits "HTTP code: phrase" where a phrase is available or "HTTP code" if not.

Sorry, but IMO the format of that string is application specific and therefore does not belong into the library. Someone else might need such a message to be formatted like "HTTP <code> - <phrase>" or "<phrase> (<code>)" or whatever.
So I will now create a separate pull request to add the missing status codes but leave out the reasonPhraseEx() function.
Let me know if you see it differently.

I completely understand and accept your reasoning.
Thank you for being so patient with me.


failed to extract shortcode: template for shortcode "icon" not found

Just updated from the academic -> starter-academic change and first thing I get when I try to run hugo in the root directory of my project is the following error message:
Start building sites …
Built in 27 ms
Error: Error building site: "/mnt/4ADE1465DE144C17/Programming/javascript/mvrozanti.github.io/themes/starter-academic/content/authors/admin/_index.md:77:1": failed to extract shortcode: template for shortcode "icon" not found

Expected behavior
Updated module is backwards compatible (?)
Maybe my old config.toml is no longer valid or something?
Versions

Link to your GitHub project: link
Wowchemy Version (from your go.mod):

module github.com/wowchemy/starter-academic

go 1.15

require (
	github.com/wowchemy/wowchemy-hugo-modules/netlify-cms-academic v0.0.0-20210127225730-4ac1c6aa1a40
	github.com/wowchemy/wowchemy-hugo-modules/wowchemy v0.0.0-20210127225730-4ac1c6aa1a40
)

Hugo Version (run hugo version): Hugo Static Site Generator v0.80.0/extended linux/amd64 BuildDate: unknown
Browser/OS:

Mozilla Firefox 84.0.2
Linux 5.10.9-arch1-1

Wowchemy Template: starter-academic

https://wowchemy.com/docs/guide/troubleshooting/#error-file-not-found-or-failed-to-extract

https://wowchemy.com/docs/guide/troubleshooting/#error-file-not-found-or-failed-to-extract

Thanks, that worked!
One more thing: I just now noticed this massive wowchemy update, is there a step-by-step migration guide from the old configuration? I tried just replacing template's content directory with my own  but that's giving me a blank homepage.

Thanks, that worked!
One more thing: I just now noticed this massive wowchemy update, is there a step-by-step migration guide from the old configuration? I tried just replacing template's content directory with my own  but that's giving me a blank homepage.

https://wowchemy.com/docs/guide/update/


Unit test fixes for changes to audit log processing

Audit log searches are now filtered over the promotion and demotion date ranges
Audit log fixture data actor changed to match report user otherwise all entries are filtered out
Also fixes a formatting issue in config.js

@droidpl - Was curious to see what the unit test changes were.  Think this covers them.

@steve-norman-rft exactly that was the thing. I knew it was related to the phrase as I changed it to get better results, and seems I forgot to change the tests.


[Prediction Bug] Apollo did not generate a predicted trajectory for a vehicle.

Describe the bug
When the ego passed through the intersection,  there was an NPC coming slowly(0.4m/s) on the left side. The ego failed to predict the trajectory of the NPC  resulting in a collision with the NPC at the intersection.
*Note: Simulator platform was LGSVL and the perception results were ground-truth data provided by LGSVL.
System information
OS Platform and Distribution: Ubuntu 18.04
Apollo installed from (source or binary): source
Apollo version : 6.0 (master branch - commit b5bd74112e2ec5d7c65cad6bea592353de566156)
To Reproduce
The video corresponding to the collision scenario is in the attachment.
Expected behavior
The ego should predict the trajectory of the NPC correctly to evade it in time.
Additional context
It seems that EmptyPredictor in the prediction module is designed for vehicles with speeds less than a certain threshold (still_obstacle_speed_threshold:0.99m/s) to output empty predicted trajectories, which leads to the failure to output the predicted trajectory for the dangerous NPC in this scene. And if we change the still_obstacle_speed_threshold from 0.99m/s to 0.1 m/s, the collision can be avoided.
Demo Video
Before changing the still_obstacle_speed_threshold
https://user-images.githubusercontent.com/87695978/193719127-6a81eb3e-aff3-47b2-9d83-58c9c1d4e5ed.mp4
After changing the still_obstacle_speed_threshold
https://user-images.githubusercontent.com/87695978/193719152-b99b544c-3493-4223-9c85-7bea049cf1d7.mp4
Attachments
Collision Video
Not-Collision Video

This is previously reported in https://github.com/ApolloAuto/apollo/issues/11104
Apollo has a threshold configuration for obstacles' speed. I think it's about 0.5m/s2.
Whenever the obstacle's speed is slower than 0.5m/s2, Apollo treats it as STATIC obstacles, and of course, it does not make trajectory predictions for "Static" obstacles.

Same as https://github.com/ApolloAuto/apollo/issues/11104 and similar to https://github.com/ApolloAuto/apollo/issues/11118


Consider using azure jet provider

While testing #1, we've had a lot of flakiness and unreliability of the AKS resource specifically.  We should consider (after merging #1) migrating the implementation to use https://github.com/crossplane-contrib/provider-jet-azure/ instead of the native provider-azure, after confirming that reliability is indeed improved.
I'd love to be able to consistently bring up all the resources in this reference platform.

We should probably consider using the jet providers for all of our reference platforms too (GCP, AWS, etc.), but Azure seems to be in the biggest need of it.

Seems Installing jet providers in various clouds native k8s services is still a bit heavy load in the beginning. But they seem to load somewhat easily in ubc. I'm going to convert a revised platform ref I'm working on for k8s/data services with jet. beginning with azure. if it works reliably, Aaron and I will start porting to AWS and GCP. This is also part of the overall standardization of platoform refs, so two birds.

@jbw976 @garreeoke
Spent more time than expected chasing some issues with jet-azure. Have those resolved and/or reported, but am now hitting another blocker. I initially set off to revamp the platfomr refs to have an app connected to the database. Figured Wordpress would be simple enough.
Challenge is that we have no good way to create the external db that Wordpress would use. provider-sql doesn't work unless we open firewall rules to the control plane. So that's when I set back toward migrating to jet.
Clusters, network, database all provisioning fine. But it appears as though the current azure jet provider has no CRDs for creating DBs either. They are documented in the API reference, but apparently not implemented in the compiled provider.

Ok, have most things configured. But with the super long waits for every kubectl command, and super long provision/deprovision times, I can't see this being viable as a showcase yet.

Our of curiousity @natereid72 - what versions of kube api-server and kubectl were you using?
and how do those compare to:

kubectl https://github.com/ulucinar/crossplane/blob/fix-2895/design/one-pager-crd-scaling.md#client-side-throttling
api-server https://github.com/ulucinar/crossplane/blob/fix-2895/design/one-pager-crd-scaling.md#api-server-resource-consumption

Our of curiousity @natereid72 - what versions of kube api-server and kubectl were you using?

kubectl 1.23.0 (so will rev to 1.23.5)
Server side was two, whatever is running in UbC today (less than 1.23.x), and then 1.23.5 on my Kind cluster. I'm sure I could get crazy perf improvement with my home server clusters, but wanting to stay inline with what people are likely to test with and what we can provide directions in a guide for.

with 1.23.5 on both ends, client side throttling is eliminated. I would call kubectl get managed less slow. It's probably within acceptable wait time for a platform ref guide. thanks for the reminder to check all the details Jared.

Most all of the provider-jet-azure is v1alpha1 now, so not ready to be used in this reference platform.  We (@natereid72) can follow up to prioritize some of the MRs to bump them to v1alpha2 with manual config, but we're not ready to build on provider-jet-azure just yet.

What is the fate of this ticket now? we need to figure out what to do with platform-refs in the context of almighty official providers.


Vector netcode

Updated the network code to be more developer friendly as other developers look at what's going on under the hood.
Additionally i changed the way the UVI reflects status's.  Now there is a Downloading, Processing, and Complete status.
Issues:  #59 #66

Found #86 and #87 while testing this. However, these are not blocking issues. Feel free to merge.


unable to upload folder to Azure file share

only file upload works:

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

ID: 4ef60665-5b74-4028-7c47-68cd6b666732
Version Independent ID: 038f9521-50f6-475e-e626-60d4127290ec
Content: Using Storage Explorer with Azure Files
Content Source: articles/vs-azure-tools-storage-explorer-files.md
Service: storage
GitHub Login: @CawaMS
Microsoft Alias: cawa

@amarkevich  Thanks for your question. We are checking on this and will respond to you soon.

@amarkevich Presently you can only  upload file through Storage Bowser (Preview)
Feel free to send feedback about the capability.
All the feedback you share in these forums will be monitored and reviewed by the Microsoft engineering teams responsible for building Azure.

@SumanthMarigowda I followed the documentation section "Upload a folder to a file share". Article should be updated in case this feature unavailable because confusing users.


Favicon

Design a favicon for the site

@touchRED figma has a few draft favicons. when sasha is back from vacation he can let us know which is final

Favicon designed and signed off by the client, ready for implementation @touchRED!
Figma can be found here


fix: cacheControl header when uploading

fixes https://github.com/supabase/storage-api/issues/18
Looks like the non file values need to be sent before the file fields in form-data as mentioned here and it is the intended behaviour.

:tada: This PR is included in version 1.0.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Wildcard routes aren't supported properly in config

I have a staticwebapp.config.json with a route defined as follows:
{
  "routes": [
    {
      "route": "/manage*",
      "allowedRoles": ["authenticated"]
    }
  ]
}

While this is how you should match a path per the docs, the CLI doesn't correctly generate the regex required to do the wildcard matching, it generates /^\/manage*$/ not /^\/manage.*$/ (not the missing . to indicate the any character after match). There also isn't a test case to cover this in the glob.spec.ts tests.
You can see the failure to find it appropriately looking at the output in the silly logging (it's about a dozen lines down):
[swa] --------------------------------------------------------
[swa] ------------------- processing route -------------------
[swa] --------------------------------------------------------
[swa] processing /manage/events
[swa] checking for matching route
[swa] check if request match route
[swa]  - route: /manage*
[swa]  - wildcard: true
[swa] checking wildcard route
[swa]  - glob: /manage*
[swa]  - pathBeforeWildcard: /manage
[swa] checking if glob expression is valid
[swa]  - glob: /manage*
[swa]  - glob ends with *. Return true
[swa]  - route regexp: /manage*
[swa] turning glob expression into valid RegExp
[swa]  - glob: /manage*
[swa]  - regexp: /^\/manage*$/
[swa]  - isMatch: false
[swa]  - alternateRequestPath: /manage/events/index.html
[swa] checking wildcard route
[swa]  - glob: /manage*
[swa]  - pathBeforeWildcard: /manage
[swa] checking if glob expression is valid
[swa]  - glob: /manage*
[swa]  - glob ends with *. Return true
[swa]  - route regexp: /manage*
[swa] turning glob expression into valid RegExp
[swa]  - glob: /manage*
[swa]  - regexp: /^\/manage*$/
[swa]  - isMatch: false
[swa] checking auth request
[swa]  - not an auth request
[swa] checking function request
[swa]  - not a function request
[swa] checking data-api request
[swa]  - not a data Api request
[swa] checking HTTP method: GET
[swa]  - method is valid (allow-list: GET,HEAD,OPTIONS)
[swa] checking for query params
[swa] checking rewrite auth login request
[swa] checking rewrite auth logout request
[swa] checking authorizations for route
[swa]  - no matching rule
[swa]  - access authorized
[swa] using userConfig
[swa] - userConfig:
[swa]  - routes:
[swa]   - 0:
[swa]     - route: /manage*
[swa]    - allowedRoles:
[swa]      - 0: authenticated
[swa]  - responseOverrides:
[swa]   - 401:
[swa]     - rewrite: /
[swa]     - statusCode: 200
[swa] checking storage content
[swa] remote dev server detected.
[swa]  - url: /manage/events
[swa]  - target: http://localhost:5175
[swa] customUrl: false
[swa] is4xx: false
[swa] remote dev server detected. Proxying request
[swa]  - url: /manage/events
[swa]  - code: 200
[swa] GET http://localhost:5175/manage/events (proxy)
[swa] Connected to tcp:<IP_ADDRESS>:5175 successfully
[swa] http://localhost:5175 validated successfully
[swa] Connected to tcp:localhost:5175 successfully
[swa] getting response from dev server
[swa] GET http://localhost:5176/manage/events - 200

Any updates on this @cjk7989 ?
Wildcards not working correctly in local development makes testing more complex route permissions impossible.

@thomasgauvin I just ran into this again in a customer scenario with local integration and e2e tests, any roadmap on this?


Added sql describe statement

I wanted to add the describe statement, it looks like it's meant to be handled as a simple SQL Result and this works just fine, but the response is just:
[('id', 'INTEGER', True, 'PRIMARY KEY', False, True), ('name', 'VARCHAR[100]', True, 'NO', False, False)].
Which is manageable as long as you know what each value is referring to but can be confusing otherwise
So that begs the question: Should this be a dict instead?
i.e.
{ 'column': 'id', 'type': 'INTEGER', 'nullable': True, 'index': 'PRIMARY KEY', 'auto_increment': False, 'unique': True }

Or should a hint or additional value be returned?

In my opinion, that should be a map OR it can be an array of dataclass row descriptor: in this way it will not position-coded which is not intuitive and error prone

I wanted to add the describe statement, it looks like it's meant to be handled as a simple SQL Result and this works just fine, but the response is just: [('id', 'INTEGER', True, 'PRIMARY KEY', False, True), ('name', 'VARCHAR[100]', True, 'NO', False, False)].
Which is manageable as long as you know what each value is referring to but can be confusing otherwise
So that begs the question: Should this be a map instead? i.e.
{ 'column': 'id', 'type': 'INTEGER', 'nullable': True, 'index': 'PRIMARY KEY', 'auto_increment': False, 'unique': True }

Or should a hint or additional value be returned?

The columns in the result set describe the meaning of each positional value, as a normal query result.
e.g. immuclient is able to just render as a table:
immuclient>describe mytable
+----------+---------------+----------+---------------+----------------+--------+
|  COLUMN  |     TYPE      | NULLABLE |     INDEX     | AUTO INCREMENT | UNIQUE |
+----------+---------------+----------+---------------+----------------+--------+
| "id"     | "INTEGER"     | true     | "PRIMARY KEY" | true           | true   |
| "title"  | "VARCHAR[10]" | true     | "YES"         | false          | false  |
| "active" | "BOOLEAN"     | true     | "YES"         | false          | false  |
+----------+---------------+----------+---------------+----------------+--------+

@jeroiraz The reason I bring this up is that within the Python SDK we go from
A response like this:
  columns: "(defaultdb.testtable1816.id)"
  columns: "(defaultdb.testtable1816.name)"
  values {
    n: 1
  }
  values {
    s: "Joe"
  }
}
columns {
  name: "(defaultdb.testtable1816.id)"
  type: "INTEGER"
}
columns {
  name: "(defaultdb.testtable1816.name)"
  type: "VARCHAR"
}

To a return value like this:
[(1, "Joe")]

Which seems to work just fine and provides a Pythonic way of handling query results; but omits the column names which is not something a describe statement should do.
My aim is to either keep both; describe and query as close as possible to each other but continue offering the same ease of use and maybe improve querying a bit without breaking anything

@SimoneLazzaris  I made some more changes, I can do the same for queries if you think it's OK but it will not be backwards compatible

I think that the describe situation is different from a simple query: in a query, you can explicitly tell the sql engine which are the column you want, and in which order, so it's reasonable to get an array of (anonymous) tuples. In the describe scenario, I want to inspect a table, and python being python, I (user) expect some degree of introspection.
So I'd like to have an explicit dataclass like this:
@dataclass
class ColumnDescription:
    name: string
    type: string
    nullable: bool
    index: bool
    autoincrement: bool
    unique: bool

and then have the result as an array of ColumnDescription instances. Or maybe provide that as an "high level" method.

I like the idea of ColumnDescription, the only difference is the index field; I've set it to str to match the response

moved dataclass over to datatypes.py and added a missing import to verifiedSet.py


Export overwrites objects from other scenes when called from operator

Hello! First of all, thank you all for the work you do in maintaining and working on the GLTF exporter - it's a real life-saver when it comes to getting data out of Blender in a comfortable and customizeable way. Really appreciate it!
As for the core issue, I'm running into this problem with my add-on that makes use of the GLTF exporter for my homemade 3D renderer, Tetra3D. I'm running Blender 3.2.1, with the packaged GLTF exporter.
When I export a GLTF file from the menu, the resulting file seems to be fine. However, when I call to export a GLTF file from the exporter operator (either triggered through GUI button press or through a save_post handler), the exporter doesn't export the right objects in the right slots (in this particular example, collection instances across multiple scenes). I believe at its core, the GLTF exporter isn't looking at the scene it should be when exporting through an operator, and may be tied to issue #1073 - I'm not sure on this one.
Here's a simple example blend file that exemplifies the issue - I have a Cube in Scene A, and in Scene B a collection instance of that same Cube.
exportTest.zip
If you export the blend file as a GLTF file from the menu, you'll get something like this:
"asset" : {
        "generator" : "Khronos glTF Blender I/O v3.2.43",
        "version" : "2.0"
    },
    "scene" : 0,
    "scenes" : [
        {
            "name" : "Scene A",
            "nodes" : [
                0
            ]
        },
        {
            "name" : "Scene B",
            "nodes" : [
                2
            ]
        }
    ],
    "nodes" : [
        {
            "mesh" : 0,
            "name" : "Cube"
        },
        {
            "mesh" : 0,
            "name" : "Cube"
        },
        {
            "children" : [
                1
            ],
            "name" : "Instanced",
            "translation" : [
                4.643479347229004,
                0,
                0
            ]
        }
    ],
...

Which is, to my understanding, correct. However, the bug arises when exporting through the bpy.ops.export_scene.gltf() operator. So, I've also packaged in the blend file a script that you can run in the text editor that will add a "test export" button to the object's Properties panel, which you can press to export the GLTF file through code (to a file named exportTest.gltf) and thereby recreate the bug. Pressing the button will export the GLTF file as before, but the contents will be incorrect and change depending on which scene you have selected.
If you have Scene A selected, you'll get this output:
    "asset" : {
        "generator" : "Khronos glTF Blender I/O v3.2.43",
        "version" : "2.0"
    },
    "scene" : 1,
    "scenes" : [
        {
            "name" : "Scene A",
            "nodes" : [
                0
            ]
        },
        {
            "name" : "Scene B",
            "nodes" : [
                1
            ]
        }
    ],
    "nodes" : [
        {
            "mesh" : 0,
            "name" : "Cube"
        },
        {
            "mesh" : 0,
            "name" : "Cube"
        }
    ],
...

Note that the instanced collection is no longer there.
If you have Scene B selected, you'll get the following output:
    "asset" : {
        "generator" : "Khronos glTF Blender I/O v3.2.43",
        "version" : "2.0"
    },
    "scene" : 1,
    "scenes" : [
        {
            "name" : "Scene A",
            "nodes" : [
                1
            ]
        },
        {
            "name" : "Scene B",
            "nodes" : [
                3
            ]
        }
    ],
    "nodes" : [
        {
            "mesh" : 0,
            "name" : "Cube"
        },
        {
            "children" : [
                0
            ],
            "name" : "Instanced",
            "translation" : [
                4.643479347229004,
                0,
                0
            ]
        },
        {
            "mesh" : 0,
            "name" : "Cube"
        },
        {
            "children" : [
                2
            ],
            "name" : "Instanced",
            "translation" : [
                4.643479347229004,
                0,
                0
            ]
        }
    ],
...

Note here that both objects have been replaced with instanced collections.
Also, in both of the above outputs, the "scene" property is incorrect and points to the last scene in the blend file, rather than whichever one you have selected when running the operator.
As an aside, in my own testing, I don't believe this is a problem if you run the operator with the mouse hovering over the 3D view, which may help to indicate where the issue may lie.

OK, so I tested again and it doesn't have to be an instanced collection - any object can be in the second scene. If you export from Scene A, then both scenes will have the cube from Scene A. If you export from Scene B, then both scenes will have the cube from Scene B.

Hello,
Can you please share the .blend file(s)
You zip contains only the glTF. I need the .blend file to reproduce your issue.
Thanks!

Whoops, my mistake! Here you go:
gltfBuggedExportOperator.zip
Just to sum up the problem demonstrated in this blend file:
When exporting a GLTF file from the menu, everything's fine.
When exporting from an operator, the objects exported for each scene are just duplicates of the ones in the current scene.
To see this, unzip the blend file and run the "Text" script. Press the "test export" button in the object panel, and examine the outputted "exportTest.gltf" file (which will be in the same directory as the blend file wherever you unzipped it). The object information in the GLTF file will be incorrect for whichever scene is not selected when exporting.

Confirmed

Hey, thanks for fixing this - I tested out the fix and it resolves the issue. Thanks a lot!


Message: mysql_errno(): XXX is not a valid MySQL-Link resource

there are many database operation on a controller. every db operation operate by clone ($_db = clone $this -> db). the page show some notice:
===============================================================================
A PHP Error was encountered
Severity: Warning
Message: mysql_errno(): 31 is not a valid MySQL-Link resource
Filename: mysql/mysql_driver.php
Line Number: 470
A PHP Error was encountered
Severity: Warning
Message: mysql_error(): 31 is not a valid MySQL-Link resource
Filename: mysql/mysql_driver.php
Line Number: 457
A Database Error Occurred
Error Number:
SELECT * FROM ....
when i delete some db operation , the page shows correct.
is there anyone meets this problem?

why do you need to clone the instance?


Python3 support?

Wonder if this module has Python3 support and what to do to adjust.
Tried getting it in my python3 pip but got the issue of:
read(README)...

Instead I used open(README)... and worked. However the new issue is that when it tries to import the coinmarketcap_api and fails.
  File "/usr/local/lib/python3.4/dist-packages/coinmarketcap/__init__.py", line 19, in <module>
    from coinmarketcap_api import (
ImportError: No module named 'coinmarketcap_api'

Hi there,
thank you for reporting this. I'll try to fix this as soon as I find some spare time.
PR's are welcome though :stuck_out_tongue:
Cheers.

@Jzarecta I've found the issue, I'm preparing a fix that'll work for both Python 2 and 3.
Should push it sometime this week.
Cheers.

Cool, can't wait to get the update to make my script work. Looking to get a table of the altcoin valuation of selected altcoins.

Any update on this?

@Jzarecta I've been quite busy the last few weeks but I'll try to cook something up. I basically know what the issues are, I just need an elegant way of fixing them.
Cheers.

From the #python chatroom they told me the best practice for backward compatiblity is using the 'six module': 'Six provides simple utilities for wrapping over differences between Python 2 and Python 3. It is intended to support codebases that work on both Python 2 and 3 without modification. six consists of only one Python file, so it is painless to copy into a project.'
https://pythonhosted.org/six/

@Jzarecta I've just pushed to @development, I haven't tested the code though. Could you give it a spin under Python 3.x and tell me how it works?
Thank you.

@Jzarecta fixed on @master and pushed to PyPI :smile:


" Class 'Imagick' not found"  how to solve

[0] ThrowableError in Pdf.php line 164     , Class 'Imagick' not found

install imagick extension.
Further may vary depending of your OS, etc.
In my case I use ppa:ondrej/php for multiple php versions under Ubuntu 22.04 and the command was
sudo apt-get install php8.2-imagick

Thank you for your help

Thanks for your help!


Fix edit button on Read the Docs with stable branch builds

What's happening?
The current logic for computing the GitHub URLs in Furo, when on RTD is:
https://github.com/pradyunsg/furo/blob/490527b2aef00b1198770c3389a1979911ee1fcb/src/furo/theme/furo/components/edit-this-page.html#L24
This differs from sphinx-rtd-theme (https://github.com/readthedocs/sphinx_rtd_theme/blob/9899ee4ee2f547f81e51297dc12317f018e62fdd/sphinx_rtd_theme/breadcrumbs.html#L39) and should be fixed to align with that.
Reproducer

Build Sphinx documentation set on RTD, with a stable branch or tag.
See the edit page button's URL -- it has the commit hash rather than the source ref.

Expectation
The generated URL should point to the source ref.
Code of Conduct

[X] I agree to follow the Code of Conduct.

Is there anything blocking changing this? Is it as simple as updating the path you linked:
diff --git a/src/furo/theme/furo/components/edit-this-page.html b/src/furo/theme/furo/components/edit-this-page.html
index c61b17b..e8dc5f7 100644
--- a/src/furo/theme/furo/components/edit-this-page.html
+++ b/src/furo/theme/furo/components/edit-this-page.html
@@ -17,7 +17,7 @@
 {% block link_not_available %}
 {# Make nice things happen, on Read the Docs #}
 {%- if READTHEDOCS and conf_py_path and page_source_suffix and github_user != "None" and github_repo != "None" and github_version and pagename and page_source_suffix %}
-  {% set url = "https://github.com/" + github_user + "/" + github_repo + "/edit/" + github_version + conf_py_path + pagename + page_source_suffix %}
+  {% set url = "https://github.com/" + github_user + "/" + github_repo + "/blob/" + github_version + conf_py_path + pagename + page_source_suffix %}
   {{ furo_edit_button(url) }}
 {%- endif -%}
 {% endblock %}


I'm getting object Object when a local storage item doesn't exist instead of undefined.

Thoughts?
Thanks!  This is unexpected. =)))
Thanks for the great code!

Ok I flushed it and now seem to get undefined. Weird. It was definitely not in Local Storage.  But some remnant of an object was hanging in simpleStorage for some reason. =)))
Not sure what that means. =)))  Going to run this through a bunch of tests.  See if I can isolate it.
Thanks for the great code!

Just post an example if you think you found a bug.


Suggestion: Add .ly.eth, .co.eth, .co.uk.eth, .com.eth, .se.eth, etc as sub-TLD's

Current auctions have seen people race to register a lot of names, I believe it would be better for everyone if a graph of every possible combination of .x.eth, .xx.eth, up to .xx.xxx.xx.xx.eth names could be registered.
Sub-TLD's would greatly help in that they allow for less profitability in squatting and speculation, the options simply will become too many with the cost too high
Furthermore, ly.eth and co.eth are such good domains, that the only reasonable solution is to make them registrars themselves

I think this is a reasonable idea to consider in future - but first we want to get significant experience with the current configuration

Excellent! I just wanted to put the idea out there. Also, it would be cool if preexisting domain owners could put a ethereum.txt file in their root with an Adress that would reserve a dresses such as .com.eth for a small fee for a limited time. I know it would be hard since it'd involve oraclize queries but

There are 1500+ DNS TLDs, and some of those TLD.eth names are already taken. Good example is .domains. Official ENS registrar DAPP is hosted at ens.domains, but .domains.eth cant be mapped, since domains is already taken.
Any mapping scheme should support all DNS TLDs current and future uniformly, may be reserve .dns.eth, and use the mapping .com.dns.eth, .domains.dns.eth?