How to switch off log message "RENDERED App to dom node with id:"

Would like to stop logging the following message in client on Production. Please let me know where to switch it off. Thanks.
RENDERED App to dom node with id: App-react-component-256e14d5-d66a-4e65-9833-862ef12b2103 with props, railsContext: Object {}

Example: https://github.com/shakacode/react-webpack-rails-tutorial/blob/master/config/initializers/react_on_rails.rb#L35
  # default is true for development, off otherwise
  config.trace = Rails.env.development?

does that fix your issues?

@justin808 My RAILS_ENV=production on Heroku. But still seeing the following:
[SERVER] RENDERED App to dom node with id: App-react-component-7679fbc0-7d29-4deb-a939-a08ec4e4bd81 with railsContext:

On the same initializer file, have you tried switching each of these to false?
# For server rendering. This can be set to false so that server side messages are discarded.
# Default is true. Be cautious about turning this off.
config.replay_console = true

# Default is true. Logs server rendering messages to Rails.logger.info
config.logging_on_server = true

@ty-shaikh
# For server rendering. This can be set to false so that server side messages are discarded.
  # Default is true. Be cautious about turning this off.
  config.replay_console = false

Doing the above removes the following:
[SERVER] RENDERED App to dom node with id:

Setting
# Default is true. Logs server rendering messages to Rails.logger.info
  config.logging_on_server = false

has no effect on the client console.

@xiaopow You must be overriding this at view helper call.
See the trace option:
https://github.com/shakacode/react_on_rails#react_component

@justin808 absolutely right. Thank you! Solved.


[not for merge] magpie check

Changelog entry
...
Changelog category

New feature
Experimental feature
Improvement
Performance improvement
Bugfix
Backward incompatible change
Documentation (changelog entry is not required)
Not for changelog (changelog entry is not required)

Additional information
...

LGTM


⚠ Bofrost.nl has degraded performance

In 16ba2d7, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1213 ms

Resolved: Bofrost.nl performance has improved in 193fa9a after 8 minutes.


Update Docs about unit testing

In the readme, write the docs how to run the test and write quick explanation about how it works. The unit testing actually has been set, you can check it at package.json

Several minor suggestions from me:

jest can actually provide a unit-test coverage, so we can actually know what is the coverage of our source code.
jest should be verbose so we can actually see the results in-depth.
While we're talking about unit tests, why don't we also check for circular dependencies with [dpdm](https://www.npmjs.com/package/dpdm)? This will help us so we would not fall into a trap of circular dependencies. They are bad for our code.
What do you think of making two test commands: test:ci to be run in GitHub Actions, and test to be run locally? test:ci will be run in #56 after it is done. In my mind, it could be something like:

"scripts": {
  "test": "jest --watch",
  "test:ci": "jest --ci --collect-coverage --verbose"
}

I figured out that I'd probably add these suggestions - but what do you think?

@lauslim12 , yes of course you can add it


Enable monasca deployment in ECP CI jobs

Add want_monasca_proposal=1 to mkcloud.conf

Testing https://ci.nue.suse.com/job/openstack-crowbar/402/

closing not applicable for ecp jobs


service set can not show the Instance number the set before [REPLACEMENT ISSUE]

The original issue
Id: 981
Title: service set can not show the Instance number the set before

could not be created.
This is a dummy issue, replacing the original one. It contains everything but the original issue description. In case the gitlab repository is still existing, visit the following link to show the original issue:
TODO

In GitLab by @luotian-github on Nov 8, 2019, 07:46
will return the instance number info

In GitLab by @luotian-github on Nov 8, 2019, 07:46
working on it

In GitLab by @luotian-github on Nov 8, 2019, 07:46
Fixed in #1040 #1049

In GitLab by @yhua123 on Nov 8, 2019, 07:46

format, need space
show instance number on bottom space
need timer refresh instance actual number

In GitLab by @liyanq528 on Nov 8, 2019, 07:46
fixed by pull request #1127

In GitLab by @cuiaq123 on Nov 8, 2019, 07:46
fixed

In GitLab by @sokril on Nov 8, 2019, 08:18
closed

In GitLab by @sokril on Nov 8, 2019, 08:18
assigned to @cuiaq123


vue-components: extract Input sub component from TextInput

We are about to start implementing the Lookup component which in it's resting state is indistinguishable from a TextInput component. Since the parts (label, input, feedback message) that the two  components share are supposed to look and behave identical, we extract them into smaller internal components that are  reusable.
Bug: T265005

LGTM, one question.

I'm trying understand the use case for this sub component. AFAICT, this is just encapsulating certain styles with a regular text input, where is this going to be used apart from this component?

I'm trying understand the use case for this sub component. AFAICT, this is just encapsulating certain styles with a regular text input, where is this going to be used apart from this component?

@itamargiv I tried to answer this question in the PR description. It's also outlined in more detail in the description of the linked task. We want to keep the existing TextInput component as small as possible while being able to reuse its parts for the Lookup component.

But wouldn't this effectively create coupling between Lookup and TextInput, in a way that is not necessarily desirable? Also, at least on a personal level I would find it somewhat confusing to have components that are not documented in the storybook.

@itamargiv, @SaiSan-WMDE, @guergana and I discussed this separately in a call, and decided that extracting this Input component is the way to go after all, but that it's important to document this, and not hide this as an implementation detail. We considered separating the TextInput into two components - the combination of input and label, and the FeedbackMessage component, but concluded that the two should stay contained in the same component.
I updated this PR to

add the new Input component to the storybook
add a note to the TextInput docs that it's using Input internally. (I briefly looked into generating this documentation automatically, but couldn't find anything)

Updated to group the two Input stories together. This is ready for chromatic review, @SaiSan-WMDE !

Hello! All good. Although for some reason I didn't expect to find wikit-Input--warning and --error in the Basic input page, but I guess these are reusable components too rather than states, right?

Hello! All good. Although for some reason I didn't expect to find wikit-Input--warning and --error in the Basic input page, but I guess these are reusable components too rather than states, right?

@SaiSan-WMDE wikit-Input--warning and wikit-Input--error are modifications of the same component, yup!


[Add a Play]: Calculator using React

Thank You!
What's the Play about?
What ReactJS concept will be used to create this play?
Are you willing to take it up for implementation?

#254

Thank You!
What's the Play about?
What ReactJS concept will be used to create this play?
Are you willing to take it up for implementation?

Add all the details plz.


Add Census Urban Areas 2020 from Census Bureau

Summary

Proposed Readable Name: Utah Urban Areas Census 2020
Proposed Internal Dataset Name: UrbanAreasCensus2020
Proposed External Dataset Name: urban_areas_census_2020
Proposed Data Category: Demographic
Data Description: Updated census Urban Areas for 2020
Source Agency/Entity: US Census Bureau
Contact: N/A
Update Frequency: Decennially

Updated Census urban areas dataset from the 2020 census.
The data should be available in
1 Check [x] all the areas where you expect the data to show up.

[x] Internal SGID
[x] Open SGID
[x] ArcGIS Online
[x] Open Data

The data is of high quality

[x] Sweeper checks have run and passed (@jacobdadams on 2023/06/05)
[ ] The minimum requirements for metadata are populated (@jacobdadams on 2022/00/00)
[ ] The data complies with our domain rules (@jacobdadams on 2022/00/00)

Where is the data source
Choose one.

[x] Internal SGID
~[ ] Farm from AGOL (agol item url)~

Action items

Assign a person who should complete the task by replacing name with their github @name.
Check [x] the box when the task is completed and add the date of completion.
~Strike~ out all items that do not apply.

[ ] Add data to the Internal SGID (@jacobdadams, completed: 2022/00/00)
~[ ] Configure forklift for Farm from AGOL (name, completed: 2022/00/00)~
[ ] Update relevant gis.utah.gov data pages (@jacobdadams, completed: 2022/00/00)
[ ] Add a Stewardship record (@jacobdadams, completed: 2022/00/00)
[ ] Complete a SGID.META.AGOLItems record (@jacobdadams, completed: 2022/00/00)
~[ ] Complete an AGOLItems_shelved record for any static or shelved item (name, completed: 2022/00/00)~

:robot: Automation validation

Assign yourself or someone to check the item by replacing name with their github @name.
Check [x] the box and add the date of verification 2020/01/01 when the task is verified.
~Strike~ out all items that do not apply.

[ ] Open SGID via cloudb (@steveoh on 2022/00/00)
[ ] ArcGIS Online via forklift (@stdavis on 2022/00/00)
[ ] Auditor ran successfully (@jacobdadams on 2022/00/00)
[ ] Open Data (@jacobdadams on 2022/00/00)
[ ] gis.utah.gov data pages (@jacobdadams on 2022/00/00)
[ ] sgid-index (@steveoh on 2022/00/00)

Notification

[x] Twitter (@steveoh)

Group Task Assignments

Check [x] the box when you have assigned all the tasks relevant to your group.

[ ] Data Team (@gregbunce)
[x] Dev Team (@steveoh)
[ ] Cadastre Team (@rkelson)

conductor results for tasks - 238

check
status

@jacobdadams has completed 1 out of 10 tasks
:no_entry:

@steveoh has completed 2 out of 4 tasks
:no_entry:

@stdavis has completed 0 out of 1 tasks
:no_entry:

@gregbunce has completed 0 out of 1 tasks
:no_entry:

@rkelson has completed 0 out of 1 tasks
:no_entry:

conductor results for tasks - 238

check
status

@jacobdadams has completed 1 out of 10 tasks
:no_entry:

@steveoh has completed 2 out of 4 tasks
:no_entry:

@stdavis has completed 0 out of 1 tasks
:no_entry:

@gregbunce has completed 0 out of 1 tasks
:no_entry:

@rkelson has completed 0 out of 1 tasks
:no_entry:

conductor results for demographic.UrbanAreasCensus2020

check
status

internal sgid
:no_entry:

meta table

- item id
:no_entry:

- item name
:no_entry:

- geometry type
:no_entry:

stewardship
:no_entry: Did not find demographic.UrbanAreasCensus2020 in the worksheet

Is this blocked or waiting on something @jacobdadams?

conductor results for tasks - 238

check
status

@jacobdadams has completed 1 out of 10 tasks
:no_entry:

@steveoh has completed 2 out of 4 tasks
:no_entry:

@stdavis has completed 0 out of 1 tasks
:no_entry:

@gregbunce has completed 0 out of 1 tasks
:no_entry:

@rkelson has completed 0 out of 1 tasks
:no_entry:

conductor results for demographic.UrbanAreasCensus2020

check
status

internal sgid
:no_entry:

meta table

- item id
:no_entry:

- item name
:no_entry:

- geometry type
:no_entry:

stewardship
:no_entry: Did not find demographic.UrbanAreasCensus2020 in the worksheet

Adding blocked because of @jacobdadams schedule

Ok, I got the drain snake out and unblocked this. All the data have been loaded into internal, just waiting on the magic to happen. The stewardship page and the datapage are just waiting on itemids from the AGOL publish phase.

@jacobdadams I think that you need to fix the meta table entry for this layer.
DEMOGRAPHICS -> DEMOGRAPHIC

Ahah, yeah. I've got that fixed.

The data values are wonky in AGOL. I'll check the sources this afternoon.

Ok, I've updated the values in internal, so I'm hoping change detection grabs them and forklift/clouddb update the published data. Once it's gone out to all the places, we should be good to close this.

we'll find out tomorrow. let's wait until conductor runs on monday to close it out

I corrected the spelling error in the stewardship sheet

conductor results for demographic.UrbanAreasCensus2020

check
status

internal sgid
:+1:

meta table

- item id
:+1:

- item name
:+1:

- geometry type
:+1:

open sgid
:+1:

open data
:+1:

arcgis online
:+1:

stewardship

- Description
:+1:

- Data Source
:+1:

- Website URL
:+1:

- Endpoint
:+1:

- Data Type
:+1:

conductor results for tasks - 238

check
status

jacobdadams has completed 11 out of 11 tasks
:+1:

@steveoh has completed 2 out of 4 tasks
:no_entry:


Consider removing functions unrelated to running command line programs

Hello!
This library has some seemingly unrelated functions also exported. This made it harder for me to understand how to use this library, and I don't think folks will be adding this library for ansi colour codes.
I think it'd be good to focus this library on running commands and not having these other features.
Thanks,
Louis

I agree.
When I get some time and motivation, I'd like to have a builder API with inspiration from Rust's std::process::Command and Deno's Deno.Command, e.g. with functions to add one or more args, set options for capturing stdout/stderr, and get various types of return information (including a simple success bool, status codes, and a full record of data).


Improvement of cudppSegmentedScan with additional CUDPP_UCHAR

Hi,

as pointed out in this thread:
http://groups.google.com/group/cudpp/browse_thread/thread/3788d4644114a6ff
i am running into trouble when using CUDPP_UCHAR as type for a segmented scan.

The corresponding line of code failing is:
CUDA_SAFE_CALL(cudaMalloc((void**) &(plan->m_blockSums[level]), numBlocks *
elementSize));

Am i doing something wrong or i that type not implemented yet? If not i would
like to ask for an additional data type for cudppSegmentedScan (and others?)
because i think a 4 byte type is way to much for indicating zeros and ones.

Regards, Ullrich

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 6 Jul 2011 at 8:25

Currently CUDPP_UCHAR (and CUDPP_CHAR) are only supported in cudppRadixSort (in
trunk) and cudppCompact (in 1.1.1).  I'm marking this issue as an enhancement
request for the 2.1 release.

Original comment by<EMAIL_ADDRESS>on 7 Jul 2011 at 8:00

Changed state: Accepted
Added labels: Type-Enhancement, Milestone-Release2.1
Removed labels: Type-Defect


Network Manager For Hosting Players

networkManager.matchMaker.CreateMatch(roomName , roomSize , true , "" , "" , "" , 0 , 0 , networkManager.OnMatchCreate);
In Unity 2017 , it requires 9 arguments , either i didn't know the scenario ,
please correct me as i got a null reference error.

thanks for solution
https://github.com/adnanbashir1993


Option to disable updates entirely

Valheim is updating almost daily, feels like, and it's really hard to keep up with keeping mods and the alternative branch of ValheimPlus working. We just want to stay on an old version but the container keeps updating on start even though I set UPDATE_CRON to blank.

Blank probably sets it back to a defualt value, you could try a specfic date far in the future.

must have for modded server

I agree completely way to disable updates in docker startup / for mods would be nice. Forcing game to stay in stable version with mods.

Is there any solution to stop update at starting through the envvars? Or should we fork the repo and propose a solution? Is it even possible to specify a version to limit repercussion on mods ?


FunctionCode23 Impl final

Hello! I'm not shure it's the best implementation to update request passed to runningtransaction, but it's working. Keep free to accept or suggest a better solution!
Regards!

I've added the Read/Write Multiple Registers function but won't be adding changes to Master so it can be repeatable (especially if it requires the user to access private properties and functions).
Repeatable transactions should only be used for read requests.
Repeatable read/write request can be easily implemented by the user:
function execReadWrite()
{
  const t = master.readWriteMultipleRegisters(0, 10, 0, prepareWriteValues());

  t.once('complete', () =>
  {
    setTimeout(execReadWrite, 1000);
  });
}

execReadWrite();


Transition are not ignoring html comments

Version
3.0.0-beta.14
Reproduction link
https://jsfiddle.net/bfag54kh/
Steps to reproduce
Put an HTML comment among Transition's children
What is expected?
It should ignore HTML comment and act as the comment is not here
What is actually happening?
It is throwing a warning
"[Vue warn]: Template compilation error: Transition expects exactly one child element or component.

If you remove the comment from the repro it is going to remove the warning

Look like it is intentional. I fixed other warn for invalid expression with v-else on beta.15.

Seems like this problem still exists in Vue 3.0.5 when using bundler.
Repro: https://codesandbox.io/s/vue-3-transition-comment-issue-chjvl


[HOLD for payment 2023-06-28] [$1000] Web - Details - Tooltip not present on split bill details side panel

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:

Split a bill with multiple accounts
Open the split bill group
Click on the header to open the details side panel showing the group members
Hover over the accounts and the avatars and observe there is a tooltip
Go back and click on the split bill IOU card to open the details side panel
Hover over the accounts and their avatars

Expected Result:
Tooltip should show up with their emails
Actual Result:
Tooltip not present on split bill details side panel
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: yes
Reproducible in production?: yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/896c10f0-52e0-49b5-9cb5-3e5fd401cf9e
https://github.com/Expensify/App/assets/93399543/b20c580c-b334-4d28-8dd0-487f95ad4d10
Expensify/Expensify Issue URL:
Issue reported by: @Nathan-Mulugeta
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1685513538554379
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~017795e0b76d1faee5
Upwork Job ID:<PHONE_NUMBER>286881792
Last Price Increase: 2023-06-12

I think this is the same as - https://github.com/Expensify/App/issues/20086 - asking

@esh-g you're right - these are different! Thanks for clarifying!

waiting on proposals

https://github.com/Expensify/App/issues/20052#issuecomment-1573716590
@eVoloshchak @Christinadobrzyn bump on proposal review 😇

Proposal
Please re-state the problem that we are trying to solve
There are no tooltips in SplitBillDetailsPage
What is the root cause of that problem?

Tooltips are not enabled. So have to fix that.
There is inconsistency in the return details of participants.
Numbers show @expensify.sms at the end.

What changes do you think we should make in order to solve the problem?
Solution:

Add showTitleTooltip prop to OptionsSelector in MoneyRequestConformationList.js.
We will make few updates in OptionsListUtils.js.

This updates make sures that code has same common params.

import * as PersonalDetails from './actions/PersonalDetails'; // + Added

... rest code

/**
 * Get the participant options for a report.
 * @param {Object} report
 * @param {Array<Object>} personalDetails
 * @returns {Array}
 */
function getParticipantsOptions(report, personalDetails) {
    const participants = lodashGet(report, 'participants', []);
    return _.map(getPersonalDetailsForLogins(participants, personalDetails), (details) => ({
        keyForList: details.login,
        login: details.login,
        text: details.displayName,
        firstName: lodashGet(details, 'firstName', ''),
        lastName: lodashGet(details, 'lastName', ''),
        alternateText: Str.isSMSLogin(details.login || '') ? LocalePhoneNumber.formatPhoneNumber(details.login) : details.login,
        icons: [UserUtils.getUserAvatarIconObject(details.avatar, details.login)], // + Changed to a common function getUserAvatarIconObject
        payPalMeAddress: lodashGet(details, 'payPalMeAddress', ''),
        phoneNumber: lodashGet(details, 'phoneNumber', ''),
    }));
}

...rest code

// + Added Function
/**
 * @param {Object} personalDetail - personalDetail of the user
 * @return {Object} commonParams - amount of the IOU
 */
function getCommonPersonalDetails(personalDetail) {
    const displayName = PersonalDetails.getDisplayName(personalDetail.login, personalDetail);
    return {
        text: displayName,
        login: personalDetail.login,
        displayName,
        participantsList: [{login: personalDetail.login, displayName}],
        icons: [UserUtils.getUserAvatarIconObject(personalDetail.avatar, personalDetail.login)], // Used a common function getUserAvatarIconObject
    }
}

/**
 * Build the IOUConfirmation options for showing the payee personalDetail
 *
 * @param {Object} personalDetail
 * @param {String} amountText
 * @returns {Object}
 */
function getIOUConfirmationOptionsFromPayeePersonalDetail(personalDetail, amountText) {
    return {
        alternateText: personalDetail.login,
        descriptiveText: amountText,
        ...getCommonPersonalDetails(personalDetail)
    };
}

/**
 * Build the IOUConfirmationOptions for showing participants
 *
 * @param {Array} participants
 * @param {String} amountText
 * @returns {Array}
 */
function getIOUConfirmationOptionsFromParticipants(participants, amountText) {
    return _.map(participants, (participant) => ({
        ...participant,
        ...getCommonPersonalDetails(participant),
        descriptiveText: amountText,
    }));
}

...rest code

We will update UserUtils.js to make sure that we have a common function to get the avatar icon object.

import * as LocalePhoneNumber from './LocalePhoneNumber'; // + Added

...rest code

/**
 * Provided a source URL, if source is a default avatar, return the associated SVG.
 * Otherwise, return the URL pointing to a user-uploaded avatar.
 *
 * @param {String} avatarURL - the avatar source from user's personalDetails
 * @param {String} login - the email of the user
 * @returns {String|Function}
 */
function getUserAvatarIconObject(avatarURL, login) {
    return {
        source: getAvatar(avatarURL, login),
        name: LocalePhoneNumber.formatPhoneNumber(login),
        type: CONST.ICON_TYPE_AVATAR,
    };
}

...rest code

export {

    ...rest code

    getUserAvatarIconObject, // + Added
};

Updating ReportParticipantsPage.js  to fix the @expensify.sms displaying for numbers.

// import CONST from '../CONST'; - Removed as not used

... rest code

/**
 * Returns all the participants in the active report
 *
 * @param {Object} report The active report object
 * @param {Object} personalDetails The personal details of the users
 * @return {Array}
 */
const getAllParticipants = (report, personalDetails) => {
    const {participants} = report;

    return _.chain(participants)
        .map((login) => {
            const userLogin = Str.removeSMSDomain(login);
            const userPersonalDetail = lodashGet(personalDetails, login, {displayName: userLogin, avatar: ''});

            return {
                alternateText: userLogin,
                displayName: userPersonalDetail.displayName,
                accountID: userPersonalDetail.accountID,
                icons: [UserUtils.getUserAvatarIconObject(userPersonalDetail.avatar, login)], // + Used a common function getUserAvatarIconObject
                keyForList: userLogin,
                login,
                text: userPersonalDetail.displayName,
                tooltipText: userLogin,
                participantsList: [{login, displayName: userPersonalDetail.displayName}],
            };
        })
        .sortBy((participant) => participant.displayName.toLowerCase())
        .value();
};

...rest code

Before Fix:
https://github.com/Expensify/App/assets/78416198/4ed9c46f-445b-496e-8a3c-a6f02e95d38b
After Fix:
https://github.com/Expensify/App/assets/78416198/4785ae10-0d69-4341-9d51-785ee74421b9
What alternative solutions did you explore? (Optional)
N/A

@eVoloshchak has to review the proposals

@esh-g's proposal looks good to me!
MoneyRequestConfirmationList.js was recently refactored, so exact prop names in proposal are outdated, but I agree with the general approach
🎀👀🎀 C+ reviewed!
cc: @cristipaval

@cristipaval @eVoloshchak PR is here: https://github.com/Expensify/App/pull/20742

@eVoloshchak https://github.com/Expensify/App/issues/20052#issuecomment-1585568107 - Please review and comment on my proposal as where the code is going wrong or right. So that i know how to give my proposals better.

@jeet-dhandha, your proposal is quite similar to @esh-g's proposal, with the exception of different changes to OptionsListUtils. Generally, the first proposal to correctly identify a root cause and to propose a fix for it - is a winning proposal. In this case time was the deciding factor, since both proposals are similar and both do resolve the issue
Nonetheless, thank you for the proposal, and excited to work with you on other issues in the future!

moving to weekly while PR is reviewed

PR was merged yesterday, will be deployed to staging soon

I don't think the BZ checklist is applicable here, this was a feature request, not a bug

@Christinadobrzyn I have sent a proposal on upwork. Here is my profile link: https://www.upwork.com/freelancers/~0186b1d7cc69656f22

So sorry for the delay on this, i don't know how I missed this payment.
Paid @esh-g for contributor + 50% speed bonus
Paid @Nathan-Mulugeta $250 as reporter
Pending hire for @eVoloshchak - you'll also have the speed bonus.
Internal job posting - https://www.upwork.com/ab/applicants/1665798844286881792/job-details
External job posting - https://www.upwork.com/ab/applicants/1665798844286881792/job-details
@eVoloshchak feel free to accept the offer when you can and I'll pay this out! Sorry again for the delay

hey @eVoloshchak I don't think I have the ability to pay you with a company card via NewDot... is that a new thing?

Ah, found this - https://expensify.slack.com/archives/C02NK2DQWUX/p1686871517189609 - let me know when you reach out for payment @eVoloshchak

Requested the payment via NewDot

Awesome @eVoloshchak! I think this GH can be closed because I think the payment happens outside my control and through our accounting team. Do you want to confirm when you get paid and I'll keep tracking this?

I think this GH can be closed because I think the payment happens outside my control and through our accounting team

Yeah, this can be closed, payment has been released, we can always open if there's a problem

Paid @eVoloshchak


export encryption_util.dart

This is used at the ServerDemoService it the most basic example and it can be confusing for newcomers not to know from where or why this is sending a pedantic error besides if your project is running flutter analyze this will fail, you can omit this rule (not recommended) but if it needs to be used in the method encryptKeyPairs from server_demo_service.dart  it should be exported.
This is the broken pedantic rule Don't import implementation files from another package.

@Arkangel12 change you have suggested is in the trunk branch.
https://github.com/atsign-foundation/at_client_sdk/blob/trunk/at_client/lib/at_client.dart


BROOKLYN-477: add “Provisioned Machine Requirements”

(We could also a link to this new page from the troubleshooting index.html. I've not done that here.)

Comments deserve to be addressed but can be done in subsequent PR.  Should not block merge given the elapsed time.


Fix hashify for types nested within hash maps

Fixes #100

@oeoeaio
Thank you so much for taking care of this one ❤
Regarding the code, everything looks good 👍
@flash-gordon are you ok with merging this one?

This looks ok, but I want to be sure this won't break existing code. I'll merge it into a separate branch and run some tests


Reduce length of toUpper and toLower

Size is reduced by one byte without hurting readability.
Maybe this isn't wanted, though, as the current example shows off that you can clone a value multiple times?

Hm, I hadn't recognized the equivalence between f:g.. and f,g..


Add the 'origin' parameter to callback function of find(), child(), parent() methods

Are you requesting a feature or reporting a bug?
feature
What is the current behavior?
It's impossible to get properties of parent element inside callback function of the find(), child(), parent() methods.
What is the expected behavior?
var mainElement = Selector("#main-element-id");
var childByIdPostfix = mainElement.find((node, index, origin) => node.id === origin.id + "_idpostfix");

Will do the docs


Component response with oob is not respected

Maybe I have some misconfig, but it seems that a hx-swap-oob is not respected in a component.
I have a component with a search input field, that is rendered from a parent template. The parent template contains a
 tag which is empty.
The response looks like (simplified):
{% load tz i18n htmx %}

  <form>

      <input {% on 'search_patient' %} type="text" class="form-control"  autofocus>
      <button
          class="btn btn-primary" id="patient-search-button"
          {% on 'search_patient' %}>Search</button>

  </form>

    {# this should be swapped into the #patient-search-result div #}

{% for patient in patient_list %}
      {{ patient.names.first.last_name | upper }},
      {{ patient.names.first.first_name }}
{% endfor %}

The second div is  on the top level of the response, as requested by htmx:

Note that out of band elements must be in the top level of the response, and not children of the top level elements.

But it is not loaded into that div#patient-search-result, but directly after the first div, like in the response.
I looked at the code, and saw that the context "could" have a hx_swap_oob key - but this is not documented and I don't know what to do.
Is this a bug, or just me not knowing what to do here?

Can you show  me the response you get from the serer when you press the button?

Looks normal: HTTP POST /__htmx/PatientSearchInput/search_patient 200 [0.11, <IP_ADDRESS>:46476]

AH, think I got it. It is another error. Whin I click the button, everything woks as intended. When I press [Enter] in the input field, the form submits automatically too, and then the error occurs, maybe because it's submitted twice somehow. Have to think about a .prevent-default or similar.
I think we can close this issue Definitely not related to djhtmx.

FYI: I had an `{% on 'search_patients' %} on both the input field and the button. So it fired twice, and this seemed to trigger a race condition.

I have to reopen this, as the original issue is not gone... the oob swap is not respected.
This is the view template:
{% extends ...%}
{% load htmx %}

{% block content %}
...
{% htmx Foo %}
...

...

{% endblock %}

and this is the  component template:
{% load htmx %}

blah

New content

This ends up in having 2 #here-to-place-oob tags in the rendered code.
How is that done correctly?


Add content-security-policy and x-frame-options to HttpHeaderNames

Motivation:
These headers can be used to prevent clickjacking.
Modifications:
Add static fields for content-security-policy and x-frame-options
Result:
Expose general useful names

Cherry-picked into 4.1 (cc1a209185)

Hmm.. why are these fields CharSequence while others are AsciiString?

@trustin because I am stupid :( Good catch let me do a followup

Fixed via 3b5dd4e9dc0b6e350d27a2628b3f37b2483fe4a9.... Thanks @trustin for the catch!


Can you please add Gemma-2 support, I can not run my gemma2 model

Hi @dilerbatu, what is your version of JetPack-L4T?   dustynv/ollama:r36.2.0 for JetPack 6 was rebuilt with gemma2 - https://hub.docker.com/r/dustynv/ollama/tags

Hi, the version of JetPack-L4T is 35.5.0. The main thing is I want to run this model inside of nanoLLM which seems faster than other LLM containers.

@dilerbatu yes NanoLLM uses MLC (also supports AWQ and HF backends).  Have not tested MLC performance for gemma2.  I've been meaning to update the MLC build for Jetson, it requires patching, debugging API changes, and testing a bunch of models.  Hope to have time for it at some point, but you can find my previous patches from the last build here:  https://github.com/dusty-nv/jetson-containers/blob/master/packages/llm/mlc/patches/3403a4e.diff


add test

Tests for https://github.com/dimagi/commcare-odk/pull/1117

This should be good to go. It's quick & dirty; I eventually ought to split the functions in XPathEvalTest out into utility methods.


Values accept Dict

Is your feature request related to a problem? Please describe.
Perhaps not a problem, but more of an inconvenience. For many use cases(e.g. measurements of landmarks in a SLAM-like system) it would be practical to be able to give in python Dicts to Values objects for use in the optimizer.
My use case is similar to the localization example on the front readme.md, but since I dynamically add and remove landmarks during the lifetime of the system, it is convenient to store them in a dict. A list can be made work, but then all the indices change when we remove landmarks.
Describe the solution you'd like
That the Values object accepts Dict in addition to List. Either by modifying the underlying code generation system to accept it, or by converting it to a list of the accessed elements from the dictionary. (And keeping track of the indices) I don't have an in-depth understanding of how the code generation works, but I would guess that the first one is more difficult and the last one slower.
Describe alternatives you've considered
My current workaround is to create a list of all the active landmarks, and a dictionary that maps to an index in this list instead of the landmark position. This admittedly works, but is not particularly elegant and leads to some extra iterations over all landmarks.

You should be able to put a Values in a Values, which should be similar to putting a Dict in a Values?  Is there a reason that doesn't cover what you need?

It could be that it cover what I need. I did try to do this, but I struggled with different errors.
How would I reference a key in a Values inside a Values?
E.g. (slightly modified from robot_2d_localization)
    landmarks = Values()
    landmarks["landmark_0"] = sf.V2(-2, 2)
    landmarks["landmark_1"] = sf.V2(1, -3)
    landmarks["landmark_2"] = sf.V2(5, 2)

    initial_values = Values(
        poses=[sf.Pose2.identity()] * num_poses,
        landmarks=landmarks,
        distances=[1.7, 1.4],
        angles=np.deg2rad([[55, 245, -35], [95, 220, -20], [125, 220, -20]]).tolist(),
        epsilon=sf.numeric_epsilon,
    )

How would i reference the key to eg. landmark_0?
When constructing a Factor accessing the key as keys=[f"poses[{i}]", f"landmarks[landmark_{j}]", f"angles[{i}][{j}]", "epsilon"], failed.
The full modified example: robot_2d_localization.txt

You should be able to do f"landmarks.landmark_{j}"

Correct, it works. Thank you.
It covers my use case, so I close this now.

If I generate keys of this kind (f"landmarks.landmark_{j}"), how can I derive factors in C++ based on the key name, rather than the key index?
template <typename Scalar> sym::Factor<Scalar> CreateOdometryFactor(const int i) {   return sym::Factor<Scalar>::Hessian(       sym::OdometryFactor<Scalar>,       {sym::Keys::WORLD_T_BODY.WithSuper(i), sym::Keys::WORLD_T_BODY.WithSuper(i + 1),        sym::Keys::ODOMETRY_RELATIVE_POSE_MEASUREMENTS.WithSuper(i),        sym::Keys::ODOMETRY_DIAGONAL_SIGMAS, sym::Keys::EPSILON},       {sym::Keys::WORLD_T_BODY.WithSuper(i), sym::Keys::WORLD_T_BODY.WithSuper(i + 1)}); }

Can you please create a new issue for your question?  This seems mostly unrelated to this issue.  It sounds like you're trying to do something like the Robot 3D Localization example, and I think the code you pasted is already from there.  In the new issue, can you describe a little more what you're trying to do?


Queries: Added<Foo> in filter position conflicts with &mut Foo in query position

Bevy version

0.6
0.7

Code
use bevy::prelude::*;

#[derive(Component)]
struct A;
#[derive(Component)]
struct B;
fn main() {
    App::new().add_system(hello_world_system).run();
}

fn hello_world_system(aaas: Query<&A, Added<B>>, bees: Query<&mut B>) {
    println!("hello world");
}

What should happen
cargo run this should print "hello world". As I never access B in aaas, there should be no conflict between bees and aaas.
What happened instead
thread 'main' panicked at 'error[B0001]: Query<&mut bevy_exclusion_bug::B, ()> in system bevy_exclusion_bug::hello_world_system accesses component(s) bevy_exclusion_bug::B in a way that conflicts with a previous system parameter. Consider using `Without<T>` to create disjoint Queries or merging conflicting Queries into a `ParamSet`.', /home/gibonus/.local/share/cargo/registry/src/github.com-1ecc6299db9ec823/bevy_ecs-0.7.0/src/system/system_param.rs:173:5

Additional information
Initially found out by @linkmauve in https://github.com/nicopap/ui-navigation/issues/21. I'm fairly certain the example worked at one point without crashing. I'll investigate further and report here.

When this worked previously, it was unsound (since changed and added are both stored inline in ComponentTicks).
Consider:
fn hello_world_system(aaas: Query<(), Added<B>>, bees: Query<(Entity, &mut B)>) {
    for (not, the) in bees {
        // 'the' now owns a `Ticks`, which contains an exclusive reference to `ComponentTicks` for `B` of `not`

        // Oh no, this line reads the `ComponentTicks` for `B` of `not`, even though an exclusive reference
        // to it exists. Undefined behaviour.
        aaas.get(not);
    }
}

That is, this is expected.

Thank you for the precision (and also insanely fast response). This is going to be difficult for me to migrate my code to account for that change. But the limitation makes perfect sense.

It's not ideal, but you can either use ParamSet, or make the second filter contained Added<B> (which returns a boolean of whether it was added). That would look like:
fn hello_world_system(aaas: Query<(&A, Entity)>, bees: Query<&mut B, Added<B>>) {
    for (a, entity) in aaas {
         if bees.get(entity).is_some() {
             // You know `a` has `B` added.
         }
    }
}

Alternatively, you can try and combine them all into one query, which might be easier.

Yeah, so I've a SystemParam (BunchQueries<'w,'s>) with a bunch of queries defined in module A (including a Query<&mut Foo>), and a system in module B that both uses that SystemParam and another query that filters with Added<Foo>. The "workaround" I decided to go with is split the BunchQueries in a collection of SystemParams so queries that are used together are within individual SystemParam, and have a more fine-grained query in module B's system (and functions defined in module A that accepted a &BunchQueries).
Though I'll probably come up later with a cleaner design.

Reopening. This represents a fundamental flaw in our access model.
We need four nested access levels:

Reads presence/absence
Reads values
Writes values
Writes presence absence

Added uses level 1, but currently requires level 2.

Why wouldn't that just re-introduce the unsoundness explained above?

I think the right solution here is to split apart the added and changed ticks. Messing with the added ticks should require level 4 permission, but requires level 3 permission as is.
Because level 1 and 4 permissions are incompatible, there's no soundness problems.


Ability to collapse incomplete challenges in Map across browser sessions

Currently, all the main sections on the map are collapsible. However, only the completed sections remain collapsed across browser refreshing.
It would be nice if the user was able to collapse incomplete non-certificate challenges across sessions-
especially in the early parts of the course where users with some knowledge would want to  skip the basic challenges and concentrate solely on the algorithmic scripting and projects.

Great idea! :)
Someone please add labels feature request & help wanted

I'm sorry, I know this is incredibly off-topic but I cannot find the
setting to turn off these emails. I'm getting flooded.
Jason Shih
(954) 303-6292
<EMAIL_ADDRESS>On Fri, Feb 19, 2016 at 12:08 PM, Dulshani Gunawardhana <
<EMAIL_ADDRESS>wrote:

Currently, all the main sections on the map are collapsible. However, only
the completed sections remain collapsed across browser refreshing.
It would be nice if the user was able to collapse incomplete
non-certificate challenges across sessions-
especially in the early parts of the course where users with some
knowledge would want to skip the basic challenges and concentrate solely on
the algorithmic scripting and projects.
—
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/7139.

@Soulzityr https://help.github.com/articles/unwatching-repositories/, search.

Ah there is no way to keep Watch but not get emails? Alright thanks!
Jason Shih
(954) 303-6292
<EMAIL_ADDRESS>On Fri, Feb 19, 2016 at 12:48 PM, Arsen Melikyan<EMAIL_ADDRESS>wrote:

@Soulzityr https://github.com/Soulzityr
https://help.github.com/articles/unwatching-repositories/, search.
—
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/7139#issuecomment-186326571
.

+1

This has been a previously discussed feature, but  since it didn't have a general consensus in the core team, was deemed to be implemented pending on future user request.
Perhaps what @dshgna sugests would imply some sort of "hide this section permanently" or "disable this" sort of idea? @SaintPeter, @FreeCodeCamp/issue-moderators what do you think?


https://discuss.drone.io down?

trying to open https://discuss.drone.io/c/how-tos I get:

yes, the server was shutdown. We moved to discourse.drone.io (hosted by discourse) last year


Extending for Bootstrap 4

First of all: Thanks for this really helpful plugin. As this functionality also isn´t part of Bootstrap 4 (although it´s really necessary for full responsive sites) here is the (short and easy) way to use this plugin with bootstrap 4:
Simply change the used CSS Classes for showing/hiding the elements to "hidden-lg-up" and "hidden-md-down":
TabCollapse.DEFAULTS = {
    accordionClass: 'hidden-lg-up',
    tabsClass: 'hidden-md-down',
    accordionTemplate: function(heading, groupId, parentId, active) {
        return  '
' +
                '
' +
                '
' +
                '
' +
                '
' +
                '
' +
                '
' +
                '
' +
                '
' +
                '
'

    }
};

I made a slightly more comprehensive update to use v4 cards instead of panels.
At the time of this writing it has not gone through extensive testing but it seems to work well for now.
For those interested, it is available here.


Fix/metadata dialog fix: initialSearch variable fix

Changes

the variable initialSearch was not being set when new collections loaded after logging in via EDL. Added in a step to set this variable when logging in in the Search.js file

@davidcolemanjpl please test this in Hitide UAT.  Thanks!

@jbyrne6 Please meet with David on how to test this.  Thanks!

HiTIDE-UI (UAT): v4.16.0-RC.5
HiTIDE Profile - v4.9.8-rc.1
L2SS-Py - v2.6.0rc3
Test Complete

HiTIDE-UI (OPS)  v4.16.0
HiTIDE Profile - v4.9.8
L2SS-Py - v2.5.0
Test Complete


Does this library also returns account transactions?

Does this library also returns account transactions?

Refunds are not supported yet in the library.


Checksums for sources do not match

Downloading the source archive from this link:
https://github.com/bitcoinxt/bitcoinxt/releases
https://github.com/bitcoinxt/bitcoinxt/archive/v0.11B.tar.gz
Gives me a sha256sum of:
f9fa35a0996d8b8b4eb20d77087b4463d84dc907124b73250cd82e81eed38139  bitcoinxt-0.11B.tar.gz
Whereas, the signed Gitian assert files claim the source sha256sum should be:
https://github.com/bitcoinxt/gitian.sigs/blob/master/0.11B-linux/gavinandresen/bitcoin-linux-0.11-build.assert
4af0c59de2c5d46ca250cb8b95877eae98298abfa52e9e90c71f7f785219aa6e  src/bitcoin-xt-0.11.0-B.tar.gz
I suspect this is because the source archive is being generated by Github, which is somehow different from the sources being used by Gitian. If possible, I would like to get the source tar.gz that has been signed.
Thank you!

If you want to reproduce the build yourself, you need to use gitian. You can't just grab the same source tarball, compile it and get the same build outputs. And if you do that (build the 0.11B branch) then you should get the identical source tarball out. Does that help?

I'll look into gitian, I'm not super familiar with it. However, I'm not trying to reproduce the same build outputs, I'm trying to make sure I'm using the same build inputs. In other words, I want to be using the same source tarball that you guys used and signed.
Thanks for your response!


About section correction

You named templete column "img-end", but in video, you wrote image-end, and because of that, you had to write additional code, to compensate that. With this, it should all line properly, one beneath another.

Awesome, thanks!


Log Message Contains All Metadata

I upgrade to <IP_ADDRESS> to get a hold of the custom MDC fields (thank you by the way)
However since, my messages in cloud watch are hard to read because they now contain all the meta data + the actual log message.
Before (version <IP_ADDRESS>):
{
"image": "[imagename]",
"instance": "i-xxxxxxxx",
"level": "WARN",
"logger": "[logger]",
"message": "[actual log message",
"thread": "[threadid]"
}
Nice and clean...
After:
{
"instance": "i-xxxxx",
"image": "[image]",
"level": "INFO",
"message": "2018-02-01 18:07:27.579 boxfuse/xxx [application] {"image":"[boxfuse image]","instance":"i-xxxxxx","level":"INFO","logger":"[logger]","message":"[THE LOG MESSAGE"}"
}
Logback configuration is as follows:
<appender name="cloud-watch" class="com.boxfuse.cloudwatchlogs.logback.CloudwatchLogsLogbackAppender">
	<config>
		<stdoutFallback>true</stdoutFallback>
		<customMdcKey>my-custom-mdc-key</customMdcKey>
	</config>
</appender>

It looks like the appender isn't being used to send the messages. Instead they are being sent to stdout and being picked up by the Go agent running outside the JVM. Now the interesting question is why...
Did anything else change in your setup?

OK, this is the culprit: https://github.com/boxfuse/cloudwatchlogs-java-appender/blob/master/cloudwatchlogs-java-appender/src/main/java/com/boxfuse/cloudwatchlogs/internal/CloudwatchLogsLogEventPutter.java#L207
This should first check whether the sending to CloudWatch Logs is enabled, before attempting the fallback.

This has now been fixed in <IP_ADDRESS>.

Superb @axelfontaine


Add Huabao protocol (ST-901A+, ST-902, ST-902W)

JT808.pdf
protocol.pdf
usermanual.pdf
https://github.com/traccar/traccar/blob/master/src/main/java/org/traccar/protocol/HuabaoProtocolDecoder.java

I was able to finally get my st-902 pointed at my own server. I think I was out of service when I attempted it before -- can these settings be changed with a USB cable?
I am seeing data in my logs.  I switched from debug-socket back to h02 -- the messages format is the same
[2023-02-25T18:26:45+00:00] *HQ,7018462432,V1,182552,A,4437.7528,N,08605.7779,W,000.00,292,250223,FBFFFBFF,000,00,0,0#
[2023-02-25T18:28:09+00:00] *HQ,7018462432,V1,182743,A,4438.0377,N,08605.4029,W,004.04,073,250223,FFFFFBFF,310,00,20502,185608458#
[2023-02-25T18:29:33+00:00] *HQ,7018462432,V1,182753,A,4438.0379,N,08605.4017,W,000.00,073,250223,FBFFFBFF,310,00,20502,185608458#
[2023-02-25T18:30:58+00:00] *HQ,7018462432,V1,183004,A,4438.5971,N,08605.1690,W,022.42,320,250223,FFFFFBFF,310,00,20502,185801238#
[2023-02-25T18:32:22+00:00] *HQ,7018462432,V1,183007,A,4438.6092,N,08605.1897,W,024.20,300,250223,FFFFFBFF,310,00,20502,185801238#
[2023-02-25T18:33:46+00:00] *HQ,7018462432,V1,183258,A,4439.2341,N,08606.5630,W,024.35,306,250223,FFFFFBFF,310,00,20502,185696520#
[2023-02-25T18:35:11+00:00] *HQ,7018462432,V1,183503,A,4439.5969,N,08607.4481,W,014.49,100,250223,FBFFFBFF,310,00,20502,185801238#
[2023-02-25T18:36:35+00:00] *HQ,7018462432,V1,183516,A,4439.5708,N,08607.3487,W,024.06,126,250223,FFFFFBFF,310,00,20502,185801238#
[2023-02-25T18:38:00+00:00] *HQ,7018462432,V1,183755,A,4439.4078,N,08606.9255,W,005.86,027,250223,FFFFFBFF,310,00,20502,185801238#

This data is not visible in the backend, so either the protocol is different, my settings are wrong, or perhaps I do not have the tracker ID entered correctly?

On the backend my Device serial number is correctly set to<PHONE_NUMBER>

Ok, the data is correct to protocol H02. Let me check if there are some additional column or value not parsed by default regular expression 👍🏿

After parking my car, several hours later a few trips appeared!
The times are wrong, so I think I need to still set my device to use UTC perhaps?
Either way I think the protocol is working, based on the data, but I will let you know tomororw after I test a bit more.

Actually, those times are too late to be UTC... It's not 9:55 UTC on 2/26 yet..

Yes, you need to setup your device with UTC timezone: https://github.com/eusonlito/GPS-Tracker#set-the-time-zone-to-utc-to-delegate-the-time-adjustment-to-the-platform

Also you can send a message (same text as SMS) from Devices > Messages. But TCP messages need wait to device wake up to apply the change.

I was able to update my Tracker to UTC -- thank you so much for your help!
I have my Vehicle set to my local timezone of America/New_York and I just started a new drive -- the time seems to be off by 1 hour, so maybe it is a Daylight savings issue?  Either way, not a big deal for now.
I will drive around for a few days and test everything out, and then maybe I can contribute a few updates too if you are open to it

Can you check the server logs if device is reporting the correct UTC time?
[UTC_DATETIME] *HQ,DEVICE_ID,V1,UTC_TIME,...

Or in database tables trip and position, the columns xxxxx_utc_at.
Anyway I will do some tests with your same Timezone because I have found other users with a similae issue: https://stackoverflow.com/questions/74588944/getting-the-difference-in-time1-hour-for-daylight-saving-time-timezones-like-a

Yes, try to disable the automatic timezone inside vehicle. Anyway I testing timezone conversions using several timezones.

Update the source before try disabling the automatic timezone because there was an bug with boolean casting on vehicle.

The times are correct now after a pull and disabling autotimezone!


dictFlatePool not helping much (lots of new flate compressor allocations)

Here is a snippet of alloc_space heap profile for program using pgzip to write ~770MB file:
> go tool pprof -alloc_space PROG mem.prof
Entering interactive mode (type "help" for commands)
(pprof) top20
66762.39MB of 67431.14MB total (99.01%)
Dropped 55 nodes (cum <= 337.16MB)
Showing top 20 nodes out of 41 (cum >= 12452.37MB)
      flat  flat%   sum%        cum   cum%
17734.77MB 26.30% 26.30% 17734.77MB 26.30%  compress/flate.(*compressor).init
...
 3473.48MB  5.15% 81.09%  3473.48MB  5.15%  github.com/.../vendor/github.com/klauspost/pgzip.(*Writer).compressCurrent
...
 2757.36MB  4.09% 89.66% 20834.23MB 30.90%  github.com/.../vendor/github.com/klauspost/pgzip.compressBlock

During compression, memory usage of program grows by ~5GB. With regular gzip, memory usage is constant during compression.
Am I reading this right that dictFlatePool is not helping as it should? Shouldn't the number of compressors be limited by the number of blocks?

@flyingmutant - thanks for reporting.
Is it possible for you to share the code on how you send data to pgzip?

I can't post the actual code, but what it does is basically:
f, _ := os.Create(path)
bufF := bufio.NewWriter(f)
gzF := gzip.NewWriter(bufF)

buf := make([]byte, 0, 512)
for _, record := range records {
        record.writeTo(buf)
        gzF.Write(buf)
        buf = buf[:0]
}

So, lots of small records, each is written to byte slice first (with strconv.AppendUint and the like), byte slice is sent to pgzip. Nothing fancy.


LazyProperties class tries to use the wrong map Entry class

This causes the build to fail in some environments.

Merged into develop.


Added config file for BetterCodeHub to ignore d3 library and example ouput file.

I've created a config file so that SIG can skip checking the D3 library and also the example output from out system.

Outstanding work darling <3

Bam.


Can't debug Blazor WebAssembly project launched via AppHost

Discussed in https://github.com/dotnet/aspire/discussions/1402

Originally posted by pnorriscd December 14, 2023
Not sure if this is the right place to put this.
In VS2022 17.9 Preview 2, when you create a new Blazor WebAssembly project, it asks if you want Aspire orchestration. If you click yes, it creates a new solution where you can't debug the WebAssembly app.
I haven't been able to find any way to debug WebAssembly run under app host.
Looking for any ideas or suggestions.

Same issue here for a .NET 8 Blazor Web project, with any 'rendermode InteractiveWebAssembly' components, I don't hit my breakpoints (where as I do if I start the project directly), VS 2022 Version 17.9.0 Preview 4.0.
May be incidental but I do see a number of 'Exceptions thrown: 'System.Net.Http.HttpRequestException' in System.Private.CoreLib.dll' and 'Exception thrown: 'System.Net.Http.HttpRequestException' in System.Net.Http.dll' when starting up the aspire appHost.

@vijayrkn can you have somebody look into this?

Chiming in here as I believe I am experiencing this issue too.
I have a mono repo, and when running aspire my backend projects hit breakpoints just fine, however my Blazor WASM (standalone, not interactive auto) front end consistently cannot load symbols.
My workaround is that I comment out my front end in my AppHost's program.cs like so
var builder = DistributedApplication.CreateBuilder(args);

//builder.AddProject<Projects.Frontend>("WASM Frontend");
builder.AddProject<Projects.BackendOne>("Backend 1");
builder.AddProject<Projects.BackendTwo>("Backend 2");
builder.AddProject<Projects.BackendThree>("Backend 3");

...

I then run my aspire AppHost project in visual studio to debug all of my back end projects.
Then in vscode, I run my blazor wasm front end with dotnet watch and attach a debugger to it manually. Only then can I debug both "types" of projects at once.

Hi
Any news about this issue ??
Tnx

Still problematic in VS 17.10.0 Preview 1.0, can't debug web assembly.

This issue is fixed pending release. Will update here once the release version is available.

Moving to Preview 5 in favor of removing milestone preview TBD.

This is fixed in the latest VS

Can anyone confirm is this was fixed on the Visual Studio side of the underlying SDK Tooling side please, I'm using Rider on Ubuntu Desktop and would be interested to know if this will work for me in the next release or if it will be and issue that will require a release from JetBrains Rider team.

I've just updated to aspire preview 5 and vs2022 preview 3 and can now hit my web assembly break points, woohoo!
It does now spin up another browser window on launch to the web project, with a different url and randomized looking port to the endpoint I've defined in aspire.
This is ok now I've figured out how to enable CORS for any localhost port on my API rather than the fixed ports I'd set before,
but it'd be useful to know if that random port number can be controlled somewhere?

Blazor WASM works now with aspire, but i have problems when i try to make an API call from Blazor WASM project. I get "failed to fetch error" when i try to access API from Blazor WASM application (both Blazor WASM and API projects are added to the aspire project):

@FritzTheCat9 At the moment the service discovery part doesn't work client side on wasm. I currently need to put the non-aspire url in the wwwroot/appsettings.json and pick it up in Program.cs (both in the client project) where I'm registering http clients.
https://github.com/dotnet/aspire/issues/1659

I'm using the following bits:
.NET SDK 8.0.300-preview.24203.14
Aspire.Hosting, Aspire.Hosting.AppHost 8.0.0-preview.5.24201.12
Visual Studio Enterprise 17.10.3 Preview 3.0
I'm trying to use InteractiveAuto with my components, and no breakpoints currently get hit. When switching back to InteractiveServer they work fine.
@mip1983 You noted you've got it working on VS2023 Preview 3 with Aspire 5 bits, I wonder if you can confirm what version of Microsoft.AspNetCore.Components.WebAssembly* nuget packages you have installed? Mine is 8.0.2 for the following:
<ItemGroup Label="Third Party">
	<PackageVersion Include="Microsoft.AspNetCore.Components.WebAssembly" Version="8.0.2" />
	<PackageVersion Include="Microsoft.AspNetCore.Components.WebAssembly.Server" Version="8.0.2" />
	<PackageVersion Include="Microsoft.AspNetCore.Components.WebAssembly.DevServer" Version="8.0.2" />
</ItemGroup>

@Antaris
On the .Web project:
<PackageReference Include="Microsoft.AspNetCore.Components.WebAssembly.Server" Version="8.0.4" />
On the Client project
<PackageReference Include="Microsoft.AspNetCore.Components.WebAssembly" Version="8.0.4" />
<PackageReference Include="Microsoft.AspNetCore.Components.WebAssembly.Authentication" Version="8.0.4" />

At this point in time I'm now on Aspire preview 6 and VS 2022 17.10.0 Preview 5.0, but it was working prior.


fix tab-bar initial status bug

Please makes sure these boxes are checked before submitting your PR, thank you!

[ ] Make sure you follow Element's Contributing Guide.
[ ] Make sure you are merging your commits to dev branch.
[ ] Add some descriptions and refer relative issues for you PR.
#2139  #2193

#2193
顺便吐个槽。。。这bug改了多少次了。。。


Important dates

Poster template https://docs.google.com/presentation/d/1D9gM-Gb3CW8cTe6GQGcpErWOmkpWLYDzbSPcn1TJ0yY/edit#slide=id.g19461511ca2_2_8
ECE661 Project Format-24FA.pdf

Have poster ready by Dec 7th


Tasks & Artifacts do not refresh

Tasks and artifacts do not refresh when created by the agent.
Need to implement:

graphql subscription class
react hook/component for receiving updates

see useChatMessageSubscription for reference

Artifacts now refresh as of #44 .  Tasks still need streaming support.


Update setup.py

In this updated version, I've added comments to explain what each section of the code does.
I've also updated the description of the package to provide more information about what it does.
Finally, I've included lists of packages required for the application and for running tests, and added an extras_require section to define optional packages for specific features.

Title is too vague.

I've also updated the description of the package to provide more information about what it does.

not true

Finally, I've included lists of packages required for the application and for running tests, and added an extras_require section to define optional packages for specific features.

not true


Modifying the position of a SplineNode only seems to change one control point

As mentioned in my other issue report, I'm working on a game with a player running wire between devices, using SplineMesh. Right now I have it set to change its end node's Position to the transform.position of the player. This almost works, but on inspection during play, I see that only one of the control points for that final node is moving, the other is stuck in place. This causes radical curvature changes as I move the player around.
There's a very good chance that I'm missing something here, and there might be a better way to do this. Is this intentional behavior? Or is there a better way to move the control point at runtime?

It's most certainly my understanding of how nodes work... I'm now having an issue where the direction seems to consistently punch through the origin, or original location, of my player, instead of the original problem. I'll just need to humbly investigate this further before I can come up with a complete issue.

The control points, unintuitively named "Direction" in SplineMesh, are not set in the node space but in the world space. Moving a node around won't move the associated control point and you need to move it as well.
Does it help?

I am somewhat aware of this now; it seems to have something to do with the
way that the control points are getting wider as I set the node position to
the relative player position, and the player walks away... however, what
I'm ultimately trying to do is, consistently, set the node position to the
player's position while keeping the control points at the same relative
angle and scale. It's been difficult to figure out how to do.
I am using the capitalized Direction and Position, rather than direction
and position, so it calls the appropriate methods; I just can't figure out
how to set them properly.
On Thu, Mar 3, 2022 at 1:44 PM Benoît @.***> wrote:

The control points, unintuitively named "Direction" in SplineMesh, are not
set in the node space but in the world space. Moving a node around won't
move the associated control point and you need to move it as well.
Does it help?
—
Reply to this email directly, view it on GitHub
https://github.com/benoit-dumas/SplineMesh/issues/54#issuecomment-1058480098,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZ33Z22G3UYJO5O2TTOWBDU6EQBTANCNFSM5P3NL7NA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

You only have to apply the same offset to the Direction than the Position.
var playerMotion = player.position - player.previousPosition;
splineNode.Position += playerMotion;
splineNode.Direction += playerMotion;

Alternatively, you could want the direction of you wire to be always oriented to the back of your player
var curvature = Vector3.distance(player.position, nextSplineNode);
splineNode.Position = player.position;
splineNode.Direction = splineNode.Position - player.forward * curvature
// remember, the direction of the vector (Direction-Position) is where the curve is heading,
// and its magnitude represents the strength of the curvature

Alternatively, you could want the control point to be "dragged" behind the player
var curvature = Vector3.distance(player.position, nextSplineNode);
if(Vector3.distance(dragged.position, player.position) > curvature){
    dragged.position = player.position + (dragged.Position - player.position).normalized * curvature;
}
splineNode.Position = player.position;
splineNode.Direction = dragged.position;

That was it. I was expecting it to be in local coordinates. Benoit Dumas,
this asset is amazing and you're the best.
This does give me an idea for feature requests, though; as direction
requiring global coordinates (by which I mean, needing the player's
position as well as the offset) works but isn't terribly intuitive. It
would be very handy to have a LocalDirection field which automates this. (I
hear "direction" and I anticipate that we're looking at an absolute vector
with its origin at the player's location, rather than a world coordinate.)
I'll drop it on the issues page, though.
On Fri, Mar 4, 2022 at 2:11 AM Benoît @.***> wrote:

You only have to apply the same offset to the Direction than the Position.
var playerMotion = player.position - player.previousPosition;
splineNode.Position += playerMotion;
splineNode.Direction += playerMotion;
Alternatively, you could want the direction of you wire to be always
oriented to the back of your player
var curvature = Vector3.distance(player.position, nextSplineNode);
splineNode.Position = player.position;
splineNode.Direction = splineNode.Position - player.forward * curvature
// remember, the direction of the vector (Direction-Position) is where the curve is heading,
// and its magnitude represents the strength of the curvature
Alternatively, you could want the control point to be "dragged" behind the
player
var curvature = Vector3.distance(player.position, nextSplineNode);
if(Vector3.distance(dragged.position, player.position) > curvature){
dragged.position = player.position + (dragged.Position - player.position).normalized * curvature;
}
splineNode.Position = player.position;
splineNode.Direction = dragged.position;
—
Reply to this email directly, view it on GitHub
https://github.com/benoit-dumas/SplineMesh/issues/54#issuecomment-1058977321,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZ33Z4NRYKOESELWWMU6PTU6HHTJANCNFSM5P3NL7NA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Things would be more intuitive if at least the Direction was named differently. It is not a direction at all, as the direction is indicated by the vector Position - Direction. Also, a vector named direction is though as a normalized one, but in this case the magnitude of the vector also carry the curvature strength information. Direction is definitely a bad name.
"ControlPoint" would be better but it make immediate sense only for users that are at least a little documented about Bezier curves. "Target" may be a better choice but it is not that intuitive and fails to match the Bezier curve vocabulary. ControlPoint is probably the best option...
Also, the asset is quite old now and renaming this could cause a lot of regression on the user side.
I like the idea of having a world and local version of it.
I don't work on splineMesh anymore but if you feel like doing it, I will gladly accept a PR.

Direction internally refers to ControlPoint, but is deprecated
ControlPoint is the new Direction
LocalControlPoint is a getter/setter for the internal control point relatively to the node's position.

I may do that; I'm just waiting until I have a more intuitive understanding
of how it works internally. The great thing about C# properties is that
it's perfectly possible to have a second property which goes by a different
name, but references the same internal field—like having .Feet, .Inches,
and .Meters all setting and returning an internal, hidden _length field. I
would suspect that this is the way to go with Direction.
Unfortunately, I still don't feel like I entirely get Direction, and what
specifically it is doing. I've got all the source code to look through,
though.
On Fri, Mar 4, 2022 at 9:17 AM Benoît @.***> wrote:

Things would be more intuitive if at least the Direction was named
differently. It is not a direction at all, as the direction is indicated by
the vector Position - Direction. Also, a vector named direction is though
as a normalized one, but in this case the magnitude of the vector also
carry the curvature strength information. Direction is definitely a bad
name.
"ControlPoint" would be better but it make immediate sense only for users
that are at least a little documented about Bezier curves. "Target" may be
a better choice but it is not that intuitive and fails to match the Bezier
curve vocabulary. ControlPoint is probably the best option...
Also, the asset is quite old now and renaming this could cause a lot of
regression on the user side.
I like the idea of having a world and local version of it.
I don't work on splineMesh anymore but if you feel like doing it, I will
gladly accept a PR.

Direction internally refers to ControlPoint, but is deprecated
ControlPoint is the new Direction
LocalControlPoint is a getter/setter for the internal control point
relatively to the node's position.

—
Reply to this email directly, view it on GitHub
https://github.com/benoit-dumas/SplineMesh/issues/54#issuecomment-1059305344,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZ33Z6NM7L65PDF5IO5U2TU6IZTLANCNFSM5P3NL7NA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


LiveServer running really slow on website root and won't close down

I have a QNAP NAS which I use as a webserver.
I have a number of projects under the main root folder
VSCode and live server runs fine if I open a project under the root folder.
However when I used the root folder as the vscode folder and on the root folder I created an index.html file , when I launched this it took ages to load, doesn't respond to changes (or if it is it is taking forever) and I couldn't stop the live server. I closed down Chrome and still couldn't close the server down.
An error bar popped up: Running the contributed command:'extension.liveServer.goOffline' failed.
Very helpful, not.
Now the NAS is open to the internet, but I can't understand why it is fine on sub folders, but not on the rootfolder.
I have rebooted my laptop and tried again, I have cleared the cache in chrom (I use an extension to do this as well).
Any thoughts or suggestions as to what I could look at.
It's not a deal breaker as I can run the page the old fashioned way. Oh I am using index,html (there is also an index.php there - but it's a QNAP thing?) uhmmm?

Sorry, I don't get the point. Live Server is a extension that creates a local server localhost. What is the link of your QNAP NAS with Live Server ?
Btw, which operating system you're using?
[This your root folder contains lot of file (may be 5K-10K), it may response slowly due to huge files. (All the change detection is made by chokidar library). There is a way to exclude files, checkout the settings section ]

Yes I know what you mean. It shouldn't make any difference, but something weird is going on.
I use win10.
Ahh i wonder if that's the problem. Thought the root directory contains only a few files itself, it contains a number of sub folders used for other web sites that contain quite a bit.
Oooh: In fact I have just noted a recursively existing phpMyadmin folder. That is phpmyadmin folder contains phpmyadmin ad nauseum. I'm deleting the subfolder. How the heck that happened???
I'll let you know if that solved the problem, but at least its a direction of attack, thanks.
regards,
Gordon
<EMAIL_ADDRESS><EMAIL_ADDRESS>On 30/09/2017 13:54:57, Ritwick Dey<EMAIL_ADDRESS>wrote:
Sorry, I don't get the point. Live Server is a extension that creates a local server localhost. What is the link of your QNAP NAS with Live Server ?
Btw, which operating system you're using?
[This your root folder contains lot of file (may be 5K-10K), it may response slowly due to huge files. (All the change detection is made by chokidar [https://github.com/paulmillr/chokidar] library). There is a way to exclude files, checkout the settings section [https://github.com/ritwickdey/vscode-live-server#settings] ]
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub [https://github.com/ritwickdey/vscode-live-server/issues/28#issuecomment-333306729], or mute the thread [https://github.com/notifications/unsubscribe-auth/AVttR9yurWkrXVJHjXJzJyhybevBuwR_ks5snjoggaJpZM4PpfWz].

Okay, so I eventually got rid of the recursive phpMyAdmin - had to use the NAS to uninstall and then restart which took longer than I expected and was fairly frightening ;),  but all okay now.
So my root and folders use up nearly 90Mg and server still runs v slow. I've had a look at the settings, but can't quite figure out how I exclude say a folder/directory. So I have a folder in my root called MathFiguredOut, how do I exclude this?
(btw the link of QNAP with liveserver is just that the QNAP NAS has a "Web" folder and this is what the apache server in the NAS looks at as the root folder. I'm just working directly with the files there from VSCode - don't worry I would do this in a professional environment, but since it's my werver what the heck, saves all that uploading and copying malarky :)
(till I get it wrong of course!)
G

Sorry. What is the 90Mg? Another things, pick any sub folders (not contains much file) & open with VSCODE. & try to start 'live server' and made some change & check whether it's working fast or not.
[I am also using Windows 10, No issue so far]

Sorry I meant MB (edited the post).
yes, it's fine on subfolders, just not root. Even on the largest and most numerous in terms of files folder.
Weird isn't it.
I suppose another thought is that the root folder is essentially a Linux directory, though it's all pretty transparent to win 10. If  it was a problem then why aren't the subfolders affected?
Anyway it's not a major issue as I've moved on...
Thanks for your responses, at least I fixed an issue I didn't even know I had :)

the root folder is essentially a Linux directory

I meant work-space folder. (I called it as Root)

at least I fixed an issue I didn't even know I had :

Is it fixed?

Not the substantive issue, I meant the issue of having multiple levels of phpmyadmin is fixed.
Regards, Gordon
<EMAIL_ADDRESS>On 1 Oct 2017, 7:12 pm, at 7:12 pm, Ritwick Dey<EMAIL_ADDRESS>wrote:

the root folder is essentially a Linux directory

I meant work-space folder. (I called it as Root)

at least I fixed an issue I didn't even know I had :

Is it fixed?
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/ritwickdey/vscode-live-server/issues/28#issuecomment-333395325

phpmyadmin?
but live server is for static files, not for 'php' or 'asp.net' or 'node.js'

Mate, what are you going on about? My root folder contains phpmyadmin as it's a real love server and contains all sorts of stuff. It's got nothing to do with anything. I don't use live server for phpmyadmin etc.
Let me try and make it clear.
I have a production server. It has a root folder (in apache it would be called .htdocs on the NAS it is called web) it contains a number of websites each in its own directory/folder. The root also has a website, because that's how it started.
If I use vscode and select one of the folders and work with my files: html, js, css and use live server everything is hunky dory and very good, much better than brackets live server as I can use the dev tools as well, so kudos.
However, if I select the root folder and work with my html and css files, itis incredibly slow to start up the web page using live server and any changes I make are equally slow to refresh.
It doesn't make any sense to me.  Why would liver server work fine on subfolders, but not on the main folder.
I suspect there is something happening in the QNAP NAS box, which hosts the web server that is interfering with the live server, but what it is or why is beyond me.
I just hoped that as the developer of live server you might have some insight into what is going on, but if not, no worries. As I said I've moved on now.
Thanks for your help.:)
G
regards,
Gordon
<EMAIL_ADDRESS><EMAIL_ADDRESS>On 02/10/2017 09:10:15, Ritwick Dey<EMAIL_ADDRESS>wrote:
phpmyadmin?
but live server is for static files, not for 'php' or 'asp.net' or 'node.js'
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub [https://github.com/ritwickdey/vscode-live-server/issues/28#issuecomment-333467435], or mute the thread [https://github.com/notifications/unsubscribe-auth/AVttRxO_0G9dymndAJbs0NMawt4P2N0Sks5soJplgaJpZM4PpfWz].

oh! Sorry for the late, I got your issue properly.
Whenever you start server from root folder, it just live reload get slow but from the sub folder it is fine. right?
One think you telll me.........  Is the all files are important to track its changes (because the performance is decreasing due to huge file) ?
Just a workaround : (Let me know it works for not)

Open the vscode from root folder.
Set up VScode settings  following..

 //add all unsupported file having in your project
liveServer.settings.ignoreFiles:  [
    "**/*.php"
],
//change live server root if
liveServer.settings.root: "/subfolder/mainFolder"

//add all unsupported file having in your project
"liveServer.settings.ignoreFiles":  [
"**/*.php"
],
//change live server root if
"liveServer.settings.root": "/subfolder/mainFolder",
-I added in the quote marks :)
not sure what I'm supposed to put for the /subfolder/mainFolder ?

I meant,
Suppose your root folder is C:\Project\ but all of your  static file is in C:\Project\project1\staticRoot\. you have also C:\Project\peoject2\**  and so on..... but all aren't containing   static file.
now if you open vscode from root (C:\Project\). It's best to track all file starting from  C:\Project\project1\staticRoot\.
If you've this situation, you can shift the root of the live server.
"liveServer.settings.root": "/project1/staticRoot/",
got it?

Sorry, I must be having a senior moment. :(
My root is \xxxnas\Web
below that I have \xxxnas\Web\project1,  \xxxnas\Web\project2,  \xxxnas\Web\project3 etc
I only want to work with the static files in  \xxxnas\Web
Sorry, I haven't responded sooner, but I have had to do other work :)

you meant, all of the static files are in \xxxnas\Web... not \xxxnas\Web\Anywhere-else\**
In this case,
try to exclude all folders from  \xxxnas\Web
{
     "liveServer.settings.ignoreFiles":  [
                "project1/***",
               "/somewhere-else/**",
               "project2/***,
                "**/*.php"
     ],
}

Please try that. let me know it works or not

Actually this issue is from chokidar 😞

chokidar is taking responsibility to raise event on change detection

:)
Excluding the subfolders and their php files seems to have done the trick. I can now work on the root folder and your server is working fine.
What is it about the php files then??

Wow! That's great. Live Server is for static files not for php. PHP will not run through Live Server.
By this "liveServer.settings.ignoreFiles" settings you're telling to Live Server - "I don't have any static files on other directory, so you don't have to track does file." 😃


meshcraft - place blocks in different directions

Using mouse normal  and bte collider

add collider
-mining_system.py
bte = Entity(model='cube',color=color.rgba(1,1,0,0.4), collider="box")# Add collider

Change checkBuild function to use mouse normal
-building_system.py

from ursina import Vec3, floor, mouse#import mouse
from mining_system import bte#import bte

  def checkBuild(_td):
    _bsite = bte.position
    _bsite += Vec3(0,-0.5,0)

      if mouse.normal:# Add mouse normal addition
        _bsite.x += round(mouse.normal.x)
        _bsite.y += round(mouse.normal.y)
        _bsite.z += round(mouse.normal.z)
    else:
        _bsite.y += 1

    x = floor(_bsite.x)
    y = floor(_bsite.y)
    z = floor(_bsite.z)
    if _td.get((x,y,z))!='g' and _td.get((x,y,z))!=None:
        print("Can't build here, sorry :(")
        return None
    return Vec3(x,y,z)

Change checkBuild function call
-mesh_terrain.py
bsite = checkBuild(this.td)#Change f call #str58

Point with the red dot on the highlighted block  and click to place
-----------Red dot height fix------------
After the subject initiation
subject.height = 1.86
subject.camera_pivot.y = 1.86 # = Player Height
it is the player controller code , it needs customization for meshcraft needs
we can make our own first person controller using default as a base later
to clean-up the main function
PS
Thank you for this fun tutorial! :)

Hello, sorry my bad, i think
subject.height = 1.86 in your code will break step up thing
but camera height can still be changed
Also collision on bte will break step up too
We can delete foot ray in the firstpersoncontroller or without collision
or make additional second highlight , what visible if there is a  block  in the shell around it (in directional axis positions around)  (second highlight)
O, and one idea, (i add it as a issue if ill make something later)
We can put blocks in the air with the little code refactor
We can create creatures what stand on the moving clouds or just jump on created blocks
We also can make living 'subset'
Like a snake, consisting of the blocks,
it moves blocks positions in the space
And i like your idea of speed building,
thanks


Chocolateysource priority behaves different on different Windows versions

Hi,
Tried to find a registered issue that was in some kind resembling mine, but didn’t find any. I have this pretty strange behavior from the chocolateysource resource.
My manifest has this in it:
chocolateysource {'Chocolatey':
ensure   => disabled,
}
For some reason on Server 2012 R2 this resource kept on changing the priority from 10 to 0 (I guess 0 is set when a repo is disabled). I remedied this in the ugly way in the past, just adding a priority setting set to 10. Ok, so far so good. Then Server 2016 came along and that ugly fix I had with “priority => 10” resulted in the resource constantly changing the value from 0 to 10, which would be more expected, maybe, then the previous behavior on 2012 R2.
So why is the 2 behaviors different, and which is the right one?
Is this a bug? Expected behavior? Me doing it wrong?

Could be a bug. However to add more to your confusion, chocolatey/chocolatey does not yet have chocolateysource, chocolateyfeature or chocolateyconfig, those are still only in puppetlabs/chocolatey. That means you will need to file a ticket at https://tickets.puppet.com/browse/MODULES.
It does seem like a bug looking at what you are stating.


Fix build failure, and miscellaneous cleanups

Currently, there's a build failure when link_swift_package() is called with package_root not ending in /. This happens because cargo:rustc-link-search is determined using string concatenation. This PR modifies it to use std::path join methods instead.
When trying to test this change, I realised that there were the following other miscellaneous issues, which I've fixed as well:

Removed Cargo.lock. This file is supposed to be committed to repo only by end binaries. Given that this crate is a library, it shouldn't be having this.
Add tests to the repo. Without this it would be difficult to continue to expand the work of this crate. cargo test now works.
Add Github Actions workflows to run cargo build and cargo test on every push to master branch, and for PRs.
Include crates.io, license and build status badges in README header. The build status would start working only after you've merged this PR.
Miscellaneous changes to code to satisfy rustfmt and clippy.

Thank you for this useful crate.

Wowow thankyou! I haven't touched this crate in a while but it's been in need of some maintenance. CI work is great, and the package_root problem was just me being silly.


Error occur from retest GitHub Actions

Description:

What issue is being seen? Describe what should be happening instead of
the bug, for example: api-testing should not crash, the expected value isn't
returned, etc.

Repro steps:

Include sample requests, environment, etc. All data and inputs
required to reproduce the bug.

Note: If there are privacy concerns, sanitize the data prior to
sharing.

Environment:

Include the environment like api-testing version, os version and so on.

Logs:

Include the access logs and the api-testing logs.

hi @yuluo-yx Please help to take a look at this error.

It looks like every comment will trigger it.

Maybe the version should be lowercase<EMAIL_ADDRESS>https://github.com/LinuxSuRen/api-testing/blob/42c4eb259beef22488085d7c728f7f71c333c71b/.github/workflows/command.yml#L23

sorry, is my question, It version is v1.0.0-RC1, I write was V1.0.0-RC1 😢😢

But why it is triggered no matter what comment is created. I think we need to add a filter. It can be triggered only the specific comment created.

The current logic is to get the comment content and run the failed task if it is /retest.
I'll implement his. 🤔

I'm not sure if GitHub Actions support that kind of filter.

I'm not sure if GitHub Actions support that kind of filter.

Let me see the documentation and study it

@LinuxSuRen I study envoy repo command.yml. I've found that I have to run a workflows anyway, and github doesn't have a hook for that.
For more information you can refer to this comment, he's not /retest but judging from the action logs, he's still executing it, but not the retest action
action:
https://github.com/envoyproxy/envoy/actions/runs/8845882918/job/24290669995
comment:
https://github.com/envoyproxy/envoy/pull/33818#issuecomment-2078967461

I'm afraid too many triggers will burn out the GitHub Actions minutes. See also the official document.
2,000 minutes per month

I'm afraid too many triggers will burn out the GitHub Actions minutes. See also the official document.
2,000 minutes per month

got it, It's a factor that really needs to be taken into account 🤔

By the way, you can make it be a webook server. And I know a free service that can deploy it. See https://zeabur.com/pricing

By the way, you can make it be a webook server. And I know a free service that can deploy it. See https://zeabur.com/pricing

tks, let me study


Greenkeeper/nodeshift 1.11.0

Coverage remained the same at 96.154% when pulling e6302febce429c9cf621c1b77ec175ad6b32b626 on greenkeeper/nodeshift-1.11.0 into<PHONE_NUMBER>596f919003924a7434b9fb14e6f47b on master.


[Errno 107] Transport endpoint is not connected // [Errno 113] No route to host

I got this error. It's from a Pi 3 iirc.
<IP_ADDRESS> - - [06/Apr/2021 22:44:46] "GET /socket.io/?EIO=4&transport=polling&t=NYf9I32&sid=sWegbfCfpW_e5LzcAAAA HTTP/1.1" 200 181 0.000476
Create Controller
[2021-04-06 22:44:49,861] ERROR in server: [Errno 107] Transport endpoint is not connected
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/dist-packages/nxbt/controller/server.py", line 145, in mainloop
    itr.sendall(msg)
OSError: [Errno 107] Transport endpoint is not connected

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.7/dist-packages/nxbt/controller/server.py", line 189, in save_connection
    itr, ctrl = self.reconnect(self.switch_address)
  File "/usr/local/lib/python3.7/dist-packages/nxbt/controller/server.py", line 357, in reconnect
    test_ctrl.connect((reconnect_address, 17))
OSError: [Errno 113] No route to host
[2021-04-06 22:44:50,373] ERROR in server: [Errno 107] Transport endpoint is not connected
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/dist-packages/nxbt/controller/server.py", line 145, in mainloop
    itr.sendall(msg)
OSError: [Errno 107] Transport endpoint is not connected

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.7/dist-packages/nxbt/controller/server.py", line 189, in save_connection
    itr, ctrl = self.reconnect(self.switch_address)
  File "/usr/local/lib/python3.7/dist-packages/nxbt/controller/server.py", line 357, in reconnect
    test_ctrl.connect((reconnect_address, 17))
OSError: [Errno 113] No route to host

I'll try to format the software as I did a lot of weird things in that I have forgotten already...
By the way. This project seems really interesting.

Thanks for your interest in the project!
By any chance, have you updated your Switch recently (within the last week)? Nintendo pushed v12 of SwitchOS which has thrown a wrench into current controller emulation techniques. If you haven't this may be another error, however, I suspect this is likely the issue. If your Switch is on v12, please refer to #18 for progress updates on support.

@Brikwerk I had issues with a previous project in the Pi1 as well. I'm not even sure it will work at all. My best option is some kind of reset of the operating system maybe...


Add serum_dex program to test fixture

Still very much a work in progress. Just want to get the pull request open

Hey @kevlubkcm added you to the mango v3 repo, I already setup something similar over there. See this PR: https://github.com/blockworks-foundation/mango-v3/pull/21
Ideally ping me on discord to catch up a bit on the recent changes :)
You can accept the invite here: https://github.com/blockworks-foundation/mango-v3/invitations


Issue related to AMQ Active-Passive

Does stompit.ConnectFailover handle AMQ Active-Passive configurations.  In our case, only the primary accepts client connection and the Passive one rejects, so heartbeat and connection always fails on the Passive.  Is there a way to stay on the Active via stompit and only switch when passive becomes active.

I haven't used Stompit with an AMQ Active-Passive configuration so I can't confirm how it's actually handled.
I guess the randomize: false setting is what you need. With this setting, the connect and reconnects to hosts in the specified server list in a round-robin fashion.
So the expected behaviour of stompit.ConnectFailover with configuration (['primary-host', 'secondary-host'], {randomize: false}) when for example the active host becomes unavailable and then later the passive host becomes unavailable, would look like this:

[Action] Connect to primary-host
[Event] primary-host goes offline
[Action] Connect to secondary-host
[Event] secondary-host goes offline
Goto step 1

Thanks for your response.  I tried the randomize:false setting, but it didn't seem to help either.  So there are actually two issues.  First, issue is missing some messages, when i use the http://gdaws.github.io/node-stomp/api/connect-failover/  in a Active-Passive scenario.   What i noticed is we are trying to connect to both hosts and during this time, if a message comes in, we are missing them.
So that being said, i removed the passive host and some messages are still missed, when Heartbeat is higher number, ex (1 second )and alwaysConnected flag is set to true.  Our case is such that messages come in sometimes frequently other times there maybe a pause in messages.  What's the preferred heartbeat value so that the connection is available and always connected ?.  Any help on how to use the API and parameters, and any insight on how to stay connected even when there is no activity would be useful as well.


Next v0.4.0 release

Important

Only works with the newest v0.24.0 version of MeiliSearch
Introduces a new way to index your data in MeiliSearch. You will need to re-index your content ⚠

Important

⚠ Introduces a new way to index your data in MeiliSearch. You will need to re-index your content ⚠
Only works with the newest v0.24.0 version of MeiliSearch

Features

Custom configuration file
Custom index name
Multiple collections in one index
Transform entries before indexation
Permission system
Error notification on failed entries indexation

Small changes

Checkbox to index instead of row
Clarity on different api keys

Custom Configuration
Inside the api/[collection]/model folder of all existing collections, there is an associated [collection_name].js file. In this condiguration file it is now possible to add information about how your collection should act with MeiliSearch.
All possible configurations are explained in the next sections.
Example
// api/restaurant/models/restaurant.js

module.exports = {
  meilisearch: {
    // parameters
  }
}

Custom Index Name
Previously, when adding a collection to MeiliSearch you had no choice on the index naming, it was the same as the collection name.
For example, when adding the restaurant collection to MeiliSearch, the indexUid inside MeiliSearch would also be called restaurant.
With this new version, it is now possible to define a custom index name in your collections model options.
// api/restaurant/models/restaurant.js

module.exports = {
  meilisearch: {
    indexName: "my_restaurants"
  }
}

Multiple collections in the same index
You can point multiple collections to the same index.
For example if you want both entries from shoes and shirts to go to the same index, they need to share the same indexName, for example clothes.
// api/shoes/models/shoes.js

module.exports = {
  meilisearch: {
    indexName: "clothes"
  }
}

// api/shirts/models/shirts.js

module.exports = {
  meilisearch: {
    indexName: "clothes"
  }
}

Transform sent data
Before, the whole entry of a collection was sent to MeiliSearch. Including all the sensitive fields about the authors for example. There was no way to decide which field to send to MeiliSearch.
It is now possible by setting a transformEntry function in the configuration file of the collection's model (lots of examples here)
Example to sanitize your field, and thus removing all sensitive data:
// api/restaurant/models/restaurant.js
const { sanitizeEntity } = require('strapi-utils')

module.exports = {
  meilisearch: {
    transformEntry({ entry, model }) {
      return sanitizeEntity(entry, { model })
    },
  },
}

For example, you can now:

Remove specific rows,
Sanitize the content of private fields
Change the content of a field

⚠ Avoid playing around with the id fields as it is possible your collections may be duplicated in an index.
Error notification on failed entries indexation
When indexing a collection by clicking on the un-checked you receive a notification when the index has been created successfully in MeiliSearch.
Nonetheless, if the entries addition fails, you receive no explanation as to why. The only information you see is that the counter stays at 0.
This release tracks the entries addition process and raises error notifications as to why documents were not able to be added.
Better permission system
Since strapi-plugin-users-permissions is not a mandatory plugin, the new permission system is very basic. Now, You need to be an admin in the admin panel to access or use any of the API routes of the MeiliSearch plugin.
Use the checkbox to add collection instead of row
Previously, when clicking on the row of a collection, indexation or removal was triggered. This was non-intuitive and has been removed. Now to add or remove a collection from MeiliSearch you need to click on the checkbox
Clarity on different API keys
It was not clear that the plugin required either the master key or the private key to be able to use the plugin. This leads to users using the same key in their front end to make search requests.
You need to use the public key in your front-end app and the private or master-key in Strapi. See keys documentation
fixes: #42
fixes: #112
fixes: #193
fixes: #218
fixes: #249
fixes: #284
fixes: #279
fixes: #191
fixes: #117
fixes: #126
TODO

[ ] #126
[x] #117
[x] #191
[x] #279
[x] #284
[x] #42
[x] #112
[x] #193
[x] #218
[x] #249
[x] #270

When are you planning on releasing this PR?

@haye-webbio Today or tomorrow. I think today but just in case it might be tomorrow

@haye-webbio Today or tomorrow. I think today but just in case it might be tomorrow

Looking forward to it! Thanks :)

bors merge

bors -r

bors r-

bors merge


Remove reference to LocalFileImageService

Summary
There is a sentence and a chunk of code referring to a LocalFileImageService but there's nothing that talks about what it's for or how it relates to unit testing.  This should just be removed or have some context added.  It has been sitting in this state for a few months so I'm opting to remove it for the time being.
Fixes #8225

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: veleek sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@ardalis @mvelosop can you please review this PR?

@mairaw Approved and should be merged. Sorry I missed this a few months ago!

Fixed "conflicts" and merged master in.

Hi @mairaw, just dropped by and saw this PR that is supposed to have "conflicts too complex to solve in the web editor" but trying in the CLI shows no conflicts at all 😣

@mvelosop, @ardalis, @mairaw - I just rebased the change ontop of the current master and pushed it in.  Please merge when you can.


Change font path via mixins.scss

Is your feature request related to a problem? Please describe.
I need to change the base path from ./files/. I assume I must use mixins.scss for that.
But I can't do this:
@include fontFaceCustom($fontDir: '/static/');

I must instead override the whole thing like this:
@include fontFaceCustom(
  $woff2Path: "/static/montserrat-latin-400-normal.woff2",
  $woffPath:  "/static/montserrat-latin-400-normal.woff"
);

Describe the solution you'd like
I'm using fontFaceCustom() rather than fontFace(), because I don't want the unicode stuff.
So:

the fontFaceCustom() mixin should also provide a $fontDir variable
OR, the fontFace() mixin should also provide a $unicodeRange: false option

Thanks for this great tool!

Thanks for the issue!
Do you mean we should adjust the mixin to be like this?
@mixin fontFaceCustom(
  ...
  $fontDir: "~@fontsource/#{$fontId}/files",
  $woff2Path: "#{$fontDir}/#{$fontId}-<%= defSubset %>-#{$weight}-#{$style}.woff2",
  $woffPath: "#{$fontDir}/#{$fontId}-<%= defSubset %>-#{$weight}-#{$style}.woff",
  ...
 )

 At least this change shouldn't cause any breaking changes I think? Whilst allowing the flexibility of just changing fontDir.

Yes - and it won't cause breaking changes.
It would allow us to override the base path for both mixins. Double flexibility.

So if you want the unicode stuff you use fontFace, and if you don't then you use fontFaceCustom.

Yeah, fontFaceCustom is pretty much a copy of fontFace without the whole unicode-map feature. I need to account for fonts with tons of subsets like this after all.

At some point it may be a good idea to just consolidate them into one mixin. And use a variable to let the user opt-in/out of the unicode stuff. Unsure if that's feasible, but would be neat.

Initial suggestion released in 4.2.0.

At some point it may be a good idea to just consolidate them into one mixin. And use a variable to let the user opt-in/out of the unicode stuff. Unsure if that's feasible, but would be neat.
Something like
@if $unicodeRange {
  // stuff from fontFace()
} @else {
  // etc.
}

That would also be backwards-compatible, and then you could deprecate the fontFaceCustom mixin at a later date.

Both mixins have different usecases which can't be merged, so I'm not too in favour that suggestion. Thanks for the issue though!

Thanks for that update.
As far as the merging goes - yes I see now there are differences between them other than unicode.