Does seem to work with doxygen 1.8.13

Is this still in maintenance?

The issue seems be due to an exception thrown while acessing empty childNodes .
text=doc[i].getElementsByTagName("field")[4].childNodes[0].nodeValue;

with the below piece of code it seems to work. Apologies for the crude js.
    if (!doc[i].getElementsByTagName("field").length) {
      continue;
    }
    if (!doc[i].getElementsByTagName("field")[0].childNodes.length) {
      continue; }
    if (!doc[i].getElementsByTagName("field")[1].childNodes.length) {
      continue;
    }
    if (!doc[i].getElementsByTagName("field")[2].childNodes.length) {
      continue;
    }
    if (!doc[i].getElementsByTagName("field")[4].childNodes.length) {
      continue;
    }

Hm, you should not have empty childNodes as long as you added the content of the searchdata.xml file to the search.html file.

FWIW, I just hit the same error.  Using Doxygen 1.8.15.  I tried adding the code from @dhanjit, which then allowed a search to work.
But then the search results are not quite right, some links work, but links to functions are malformed.  For example, a search result with a link to ms_gswap2(void *data2) uses a link of:
.../gh-pages/(void *data2)

which is a 404.
Also, the search results page contains an apparently wayward and non-functional text input box right above "Search Results".  See attached screen shot, taken when mouse was hovering over the link to ms_gswap2().

Thanks for reporting. I'll update my Doxygen install in a couple weeks, so I'll have a proper look at this issue then.

Hi, has there been any progress on this issue? I'm on Doxygen 1.8.15 and would love to use this!

Hi, has there been any progress on this issue? I'm on Doxygen 1.8.15 and would love to use this!

Hi @mollieswenson , I'm afraid I won't doing much maintenance on this repo as I've fully switched to asciidoc now. I found doxygen way too unreliable unfortunately.
That being said, if you or someone else wants to propose a patch, I'd be happy to push it here.

Hi @mollieswenson , I'm afraid I won't doing much maintenance on this repo as I've fully switched to asciidoc now. I found doxygen way too unreliable unfortunately.
That being said, if you or someone else wants to propose a patch, I'd be happy to push it here.


Fix: Fixed double file extension in zip

Earlier, files inside the zip file (when we are downloading all) was being name with double extension (#9 )
Before:

Now it is fixed

hey thanks for the PR. it looks like the fix though ends up trimming too much from the name

im trying to think of why im adding the additional format in the first place if the name already has the extension... perhaps you can think of a situation where the name wouldn't include? if they were uploading a file without the extension? but i would consider that an edge case
i think a better route may be to:

continue with the split, but only pop off the last segment, which would be the format, then rejoin (the risk is we're assuming there's a format on there
a better route may be to check if the format that will be applied already exists

let downloadName = name;

if ( !downloadName.endsWith(`.${format}`) {
downloadName = `${downloadName}.${format}`
}

wdyt?

yes... my commit does not work that fine.. but checking name ending with format is not also bug-proof. Sometimes when we upload file with jpeg extension (for example: test1.jpeg). we are getting jpg as extension.


Finished Seller Details Screen and Updated NavBar

Finished the structure of the seller details and updated navbar

Harry, for any new commits try merging locally to main and then doinig a pull request. Also please add package-lock.json to your .gitignore


fix(package): add back files field with src and dist

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


What if my encoder/decoder/codec depends on some configuration options, am I forced to manually wire up?

I have types that must target multiple versions of a target JSON format, and I am wonder if I can use Fleece without being forced to drop down to using combinators to manually wire things together.
Lets say I have the following:
type Name = Name of Parts: string list
   member ToJson (options:MorphirJsonFormatOptions) = failwith "TODO: encode using options"
and MorphirJsonFormatOptions =
| Classic of Version:int
| NextGen of
and NextGenVersionOptions =  {Version:Version ; NameEncodingMode: NameEncodingMode}
and Version = Version of Major:int * Minor:int * Patch:int
and NameEncodingMode =
| ArrayStyle
| SnakeCasedSymbol

Are there any recommendations how I can still benefit from the SRTP resolution?

I think this question is quite interesting. The question about how an API should be able to decode/encode different versions with a sort of ambient context such as options that you want to thread into the codecs.


[CELEBORN-257][IMPROVEMENT] Avoid one hash searching when process message in TransportResponseHandler

What changes were proposed in this pull request?
Avoid one time of search hash key when handling response.
Why are the changes needed?
Does this PR introduce any user-facing change?
How was this patch tested?

ping @waitinfuture @FMX


Test with our images from the build

Signed-off-by: Matthias Wessendorf<EMAIL_ADDRESS>

/retest

/retest

/close

/close

/reopen

/reopen

/refresh
/test all

/refresh
/test all


Feat 668 switch graph axis

Pull Request instructions:

Please follow the update protocol
Answer the questions below in detail. Your responses will be emailed to experimenters.
If the experimenters must do anything new, provide detailed step by step instructions on the wiki
If computer maintainers need to manually update anything, provide detailed step by step instructions
Use markdown syntax in order for your comments to be rendered reliably in the email: "1." instead of "1)", use four spaces for indents.
If you use the keyword "skip email" in the title, it will skip the email updates
Merges from "develop" into "production_testing" should use the keyword "production merge" in the title for reliable indexing of updates
Merges from "production_testing" into "main" should use the keyword "update main"

Describe changes:

switch position of L and R on graph

What issues or discussions does this update address?

#668

Describe the expected change in behavior from the perspective of the experimenter

position of L and R are flipped on graph

Describe any manual update steps for task computers

None

Was this update tested in 446/447?

[ ] 446

Here's what it looks like now which is flipped from current implementation on main

This switches the labels, but not the plot itself

Ahh okay my bad I thought it was just switching axis labels. I'll work more

@micahwoodard
Yes, the content to be visualized has to be flipped, not only labels.
Note, the data to be saved for analysis should be kept as is! Down-stream analysis convention shouldn't be changed.

@micahwoodard You might be able to do something as simple as plt.gca().invert_yaxis()

@alexpiet  and @hagikent I think it should be fixed now. Per Alex's suggestion, I just flipped the y axis orientation instead. Here is what it looks like now.


updating af2 and getting typing error

in ionic 2 project everything working fine but when updating the angularfire2 getting typing error :/
npm install firebase angularfire2@next --save

don't forget
$ typings install file:node_modules/angularfire2/firebase3.d.ts --save --global && typings install
Angularfire2/Installation and Setup

@shamrozwarraich @tomasvio
Actually, thefirebase3.d.ts are the problem. The latest Firebase SDK ships with typings, which we have switched to in the pre-release.
If you're using the pre-release (@next) version, remove the firebase3.d.ts from either your tsconfig.json or typings.json file.

@davideast, I've been trying to get AngularFire2 to work since yesterday by following the steps mentioned. But I'm just not able to finish STEP 1 : Installation and Setup. Something or the other keeps breaking.
Step 1 :
Used ng new angular-af2 to create a new project using the Angular CLI webpack version angular-cli: 1.0.0-beta.11-webpack.2 ( I had to use RC5 because RC4 didn't have BrowserModule in angular core if I remember correctly)
It works fine and I'm able to do ng serve
Step 2 :
npm install firebase angularfire2 --save fails due to unmet peer dependency.
So I run npm install firebase angularfire2@next --save. This solves all the angular dependency issues but leaves me with Typescript issue.

Manually changing it in the package.json doesn't yield any good results. Here is the screenshot of what happens by changing it to 1.8.0 or 1.9.0-dev.

Step 3 :
If I were to ignore these errors and move on to
typings install file:node_modules/angularfire2/firebase3.d.ts --save --global && typings install
This also fails since there is no firebase3.d.ts file present. ( And as you mentioned above, we don't need this file anymore.)
So skipping all the steps, if I directly add
import { AngularFireModule } from 'angularfire2'; in my app.module.ts and do ng serve I get the following series of errors.
ERROR in [default] /var/www/webops/angular-af2/node_modules/angularfire2/angularfire2.d.ts:13:65
Cannot find namespace 'firebase'.
So finally what should I do to get my setup running with RC5?
Your help will be appreciated! :)

@yashmurty see #425

@danielgek Interesting that works for non 2.0 projects, I'll check that out!
@yashmurty Try install TypeScript 2.0+ either:
npm install -g<EMAIL_ADDRESS>#or
npm install -g typscript@next

@danielgek Thank you very much. It fixed my problem Cannot find namespace 'firebase'.
@davideast I successfully executed the typescript 2.0 installation. I no longer receive the UNMET PEER DEPENDENCY issue. I just get some warnings on running npm install from project root.
npm WARN optional Skipping failed optional dependency /chokidar/fsevents:
npm WARN notsup Not compatible with your operating system or architecture<EMAIL_ADDRESS>npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| ^1.9.0-dev but none was installed.
But I think I can ignore that? Also, even with the Cannot find namespace 'firebase'. issue, I was still able to run the AngularFire functions from angularfire2 as given in the examples. Which is surprising because I thought it depended on Firebase and would break.

@yashmurty Those npm WARN warnings are because you are on a windows machine, but unless you're running karma in the AF2 project itself you won't run into any issues. There's also a workaround, but that's not necessary now.
The namespace issue should have been because you were on <2.0. Are you still getting it with the following import?
import * as firebase from 'firebase';

what change occur in new af ? not show in changelog ... :/

We don't create changelogs for pre-releases. Once we get enough feedback on it we will release to main and create a changelog.

@danielgek and @yashmurty, could you explain what exactly you did that got rid of the "Cannot find namespace 'firebase'" issue? I've read and tried all the suggestions in this and the other thread, and still getting the issue.

@RedFour, I read you comments on the thread #425. Apparently it's a Windows OS problem?
Apart from the solution of adding firebase to the types array of tsconfig.json, you could try importing Firebase in your app.module.ts file by typing this.
import * as firebase from 'firebase';

Thanks, this got rid of the errors for me!

@yashmurty Same problem on win10, this solved it, Thanks! You should add this to the docs!


[3.5.0] Cannot stringify a function

Environment

Operating System: Darwin
Node Version:     v18.15.0
Nuxt Version:     3.5.0
Nitro Version:    2.4.0
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      alias, pwa, app, runtimeConfig, css, build, modules, vite, experimental, nitro
Runtime Modules<EMAIL_ADDRESS>()
Build Modules:    -

Reproduction

Describe the bug
For some unknown reason I'm getting:
500
Cannot stringify a function
at flatten (./node_modules/devalue/src/stringify.js:166:43)
at flatten (./node_modules/devalue/src/stringify.js:60:39)
at flatten (./node_modules/devalue/src/stringify.js:102:15)
at flatten (./node_modules/devalue/src/stringify.js:166:43)
at flatten (./node_modules/devalue/src/stringify.js:60:39)
at flatten (./node_modules/devalue/src/stringify.js:60:39)
at flatten (./node_modules/devalue/src/stringify.js:166:43)
at flatten (./node_modules/devalue/src/stringify.js:166:43)
at flatten (./node_modules/devalue/src/stringify.js:166:43)
And it doesn't give more details, I am only passing a simple object with strings and arrays to the nuxtApp.payload ... What else could be?
Additional context
No response
Logs
No response

For some reason a component ref is being added into the payload?, I have a component that is like:
https://stackblitz.com/edit/github-w3ggbn?file=app.vue,components%2FTest.vue
In the repo works well, but in my project I get:
https://i.imgur.com/d0cJ7RW.png
(I think is the pinia store using setup that is causing the problem...)

same issue

Yes, that's very likely. The solution is probably pretty straightforward if you can provide a reproduction... 🙏
Until then you can workaround this way:
export default defineNuxtConfig({
  experimental: {
    renderJsonPayloads: false
  }
})

Seems to be coming from Pinia (as also noted by ElYaiko)

This could be a reproduction - https://stackblitz.com/edit/github-baazqd?file=app.vue (in my app, for eg, this ref is rather a Vue inject)

We didn't support passing functions in the payload previously - so that would make this a feature request (though it's already possible with a custom 'function' type defined with definePayloadReducer and definePayloadReviver.
But I get the feeling from the original issue that the reporter felt this was a regression.

This could be a reproduction - https://stackblitz.com/edit/github-baazqd?file=app.vue (in my app, for eg, this ref is rather a Vue inject - this should be on client-side though so its assigned in onMounted)

Yes, using renderJsonPayloads: false or onMounted fixes the error so the pinia's function is defined only in client-side.
Thanks everybody for your help.

I have the same issue. Working application then udpated to nuxt 3.5.2. We use pina everywhere export the actions and state in two objects. like:
const state = ref({
    userAccount,
    isUserAccountFormModalOpen,
    isUserAccountUpdating,
    userAccountLoadingError,
    isLoadingUserAccount,

    actionTarget,
    isLoading,
  })

  const actions = reactive({
    reloadUserAccount,
    updateUserAccount,
    openUserAccountModal,
    closeUserAccountModal,

    sendChangePasswordNotification,
    enableMfa,
    disableMfa,
  })

When using renderJsonPayloads: false the error message is gone but i still see it on server side. Store actions are still not working on page. Get an error: TypeError: $setup.actions.openUserAccountModal is not a function
Do I need to setup definePayloadReducer ? Don't understand that. Any other workaround we could do? Don't want to downgrade nuxt as new Vue features are awesome.

I still facing same issue today

Facing the same issue!

@RomanSkrypnik See if you can reproduce the issue. You're probably passing a non JSON-ifiable object in useAsyncData or in a pinia store -> the browser.

@danielroe it doesn't matter pinia or something else
In my case, it happens when I try to assign new value to an object:

And then I face this:

It works fine if it executes on client side
Here is my repro: https://github.com/RomanSkrypnik/nuxt-devalue-issue

@RomanSkrypnik It looks to me like you are assigning a function into your state, which Nuxt is passing across to the client. But a function can't be converted to JSON and back again, which is why you are getting 'cannot stringify a function'.

@danielroe now I see
Thanks for your answer!

Thank you! I learned about 2 ways to solve the problem.
1/ this is not preferable
export default defineNuxtConfig({
experimental: {
renderJsonPayloads: false,
},})
2/ this is right
I passed from useFetch refresh and tried to write it to actions in state
I removed the refresh and everything was fine
Thank you!

So I've added @danielroe's suggestions to my Nuxt config.
...but now I'm getting this error:

Does anyone have any ideas about this issue please?

perhaps you are using a plugin that is being processed on the server. he can't get the window
https://stackoverflow.com/questions/54979644/window-is-not-defined-in-nuxt-js


Men's BBall Realtime Data

Does data from the mbb_loaders.py file get pulled in realtime (such as load_mbb_player_boxscore and load_mbb_team_boxscore functions)? I see it's pulling from CSVs, but I don't see any data from the 2024 season yet. Is there another function I should use?

Also, hoping to see 2024 season data in here soon.


[Bug]: Error Handling in Data Loading

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
When the Error Message Shows
The error messages will be displayed in the following scenario:
Error During Data Loading:

If the file exists but there is an issue when attempting to read the data (within the try block), the following scenarios might occur:

The file is in an unsupported format (e.g., it’s not a valid CSV).
The file is corrupted or malformed.
There are permission issues that prevent reading the file.

In this case, the error message will indicate that an error occurred while loading the data, providing more context such as: An error occurred while loading data: specific error message from the exception
Conclusion
This error handling mechanism ensures that users receive clear and informative feedback, allowing them to diagnose and rectify issues efficiently.
Add ScreenShots

What browsers are you seeing the problem on?
Chrome
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

Hi, I am interested in working on this project, please assign me, and I'll share my solution as soon as you ping me !!


Trouble using centralized dependency versions in buildSrc plugins and buildscript classpath

The new centralized dependency versions feature preview in Gradle 7 is pretty great, but there are a couple of places where libs doesn't resolve.

In plugins in buildSrc, e.g. buildSrc/src/main/kotlin/MyPlugin.kt:
class MyPlugin : Plugin<Project> {

    override fun apply(project: Project): Unit = project.run {

        dependencies {
            implementation(libs.groovy)
            testImplementation(libs.junit)
            [...]
        }
        [...]
    }
}

The error I get ise: .../buildSrc/src/main/kotlin/MyPlugin.kt: (21, 32): Unresolved reference: libs

In dependencies/classpath in the root build.gradle.kts:
buildscript {
    [...]
    dependencies {
        classpath(libs.gradleVersionsPlugin)
        [...]
    }
}

Error is Script compilation error: Line 27:  classpath(libs.gradleVersionsPlugin) Unresolved reference: libs

In the root settings.gradle.kts:
plugins {
    id("com.gradle.enterprise").version(libs.versions.gradleEnterprise.get())
}

Throws approximately the same error as in pt. 2.

In the plugins block in buildSrc/build.gradle.kts:
plugins {
    id("org.jmailen.kotlinter").version(libs.versions.kotlinter.get())
}

Throws approximately the same error as in pt. 2.

Is there a way to solve any of these?
SEO: enableFeaturePreview("VERSION_CATALOGS")

There are multiple issues here, it's better to create separate issues for each. However I don't think there's an immediate action to be taken. A follow-up for plugins (4) will be implemented. Please refer to my explanation on the forums: https://discuss.gradle.org/t/trouble-using-centralized-dependency-versions-in-buildsrc-plugins-and-buildscript-classpath/39421/2?u=cedricchampeau


Update README with more details

Updated the README file to better outline how to contribute to the repository and what the website is all about.
cc: @moveson for review and spell checking 🎃

could import the hardrock2015.xls file as a starting point?

@moveson I believe they get a bunch of data in there local environment like the hardrock2015 from running rake db:seed which automagically gets run in the command rake db:setup

Looks great. Thank you!

Ooops I think you were right. Going to open another PR to add a link to the spreadsheet and also change the actual download link over to raw. stay tuned..


Initiate communication between proximity vehicles

Create a communication flow, between ego vehicle and smart vehicle in proximity, using TCP/IP./
Output:
2024-11-19 17:52:33,253 - INFO - Received data from Smart Vehicle 24: {'id': 24, 'position': {}, 'speed': 0.0}

updated code at commit 9b493fc
output:
2024-11-19 18:07:33,837 - INFO - Received data from Smart Vehicle 98: {'id': 98, 'position': {'x': -110.9637451171875, 'y': 59.689353942871094, 'z': 0.5951882600784302}, 'speed': 0.2659561038017273}

updated code at commit aee7126
output:
2024-11-19 19:46:46,820 - INFO - Received data from Smart Vehicle 264:
2024-11-19 19:46:46,820 - INFO -   Global Position: {'x': -92.70047760009766, 'y': -53.44832229614258, 'z': 2.285251156334749e-26}
2024-11-19 19:46:46,820 - INFO -   Global Rotation: {'roll': 0.0, 'pitch': 0.0, 'yaw': -39.84444046020508}
2024-11-19 19:46:46,821 - INFO -   Speed: 0.00 m/s
2024-11-19 19:46:46,821 - INFO -   Relative Position: {'x': -19.29280409513011, 'y': 3.7019417096852125, 'z': 2.2381880536929337e-26}
2024-11-19 19:46:46,821 - INFO -   Relative Yaw: -37.07 degrees

updated code at commit 490ae1c
the lidar data from the smart_vehicle is received by the ego_vehicle, when in proximity.
Output:
2024-11-21 17:19:46,898 - DEBUG - LIDAR data processed for vehicle 123: 850 points.
2024-11-21 17:19:46,925 - DEBUG - Smart Vehicle ID 123 is still within proximity of Ego Vehicle (ID: 124) at 3.89m.
2024-11-21 17:19:46,929 - INFO - Vehicle ID 123 is 3.89m from Ego Vehicle.
2024-11-21 17:19:46,929 - DEBUG - LIDAR data processed for vehicle 123: 962 points.
2024-11-21 17:19:46,952 - DEBUG - Smart Vehicle ID 123 is still within proximity of Ego Vehicle (ID: 124) at 3.89m.
2024-11-21 17:19:46,958 - INFO - Vehicle ID 123 is 3.89m from Ego Vehicle.
2024-11-21 17:19:46,958 - DEBUG - LIDAR data processed for vehicle 123: 858 points.


Using multiple processes causes errors when retrieving active_run from MLflowTracker

System Info
- `accelerate` version: 0.17.1
- OS: Ubuntu 22.04
- Python Version: 3.10.10

Information

[X] The official example scripts
[X] My own modified scripts

Tasks

[ ] One of the scripts in the examples/ folder of Accelerate or an officially supported no_trainer script in the examples folder of the transformers repo (such as run_no_trainer_glue.py)
[ ] My own task or dataset (give details below)

Reproduction
I'm currently using MLflowTracker to track my experiments and wanted to save the metadata for each run as follows:
info = tracker.active_run.to_dictionary()
with open(file=$PATH, mode="w") as f:
    json.dump(obj=info, fp=f, indent=2)

This works fine when I'm using 1 process, but when I try to run more than 1 process then I get an error message AttributeError: 'MLflowTracker' object has no attribute 'active_run'. What's funny is that when I add a print(tracker.active_run) I get the information printed out to stdout just fine.
Expected behavior
The metadata being saved as it does when --num_processes=1.

Turns out that only one of the processes' trackers has the attribute active_run. I added a print(dir(tracker)) and got the following:
['__class__', '__delattr__', '__dict__', '__dir__', '__doc__', '__eq__', '__format__', '__ge__', '__getattribute__', '__gt__', '__hash__', '__init__', '__init_subclass__', '__le__', '__lt__', '__module__', '__ne__', '__new__', '__reduce__', '__reduce_ex__', '__repr__', '__setattr__', '__sizeof__', '__str__', '__subclasshook__', '__weakref__', 'finish', 'log', 'main_process_only', 'name', 'requires_logging_directory', 'store_init_configuration', 'tracker']

['__class__', '__delattr__', '__dict__', '__dir__', '__doc__', '__eq__', '__format__', '__ge__', '__getattribute__', '__gt__', '__hash__', '__init__', '__init_subclass__', '__le__', '__lt__', '__module__', '__ne__', '__new__', '__reduce__', '__reduce_ex__', '__repr__', '__setattr__', '__sizeof__', '__str__', '__subclasshook__', '__weakref__', 'active_run', 'finish', 'log', 'main_process_only', 'name', 'requires_logging_directory', 'store_init_configuration', 'tracker']

After doing some more investigation it seems like there's a main_process and non-main_processs. I changed my code to:
if accelerator.is_main_process:
    info = tracker.active_run.to_dictionary()
    with open(file=$PATH, mode="w") as f:
        json.dump(obj=info, fp=f, indent=2)

and it works now.

cc @muellerzr

This solution is indeed correct, nice work @seanswyi


update advanced ui web app styling in input pages 1,2,3,4 and css file

update advanced ui web app styling in input pages 1,2,3,4 and css file

The default button does not work well with numbers except for 20000

For small total population such as 200, percentages in 2dp might not work well due to mathematical rounds - individual treak might be required - I would consider it as a small task for end-users - even non-technical ones


[WIP] [ENH] Add section numbers

This is a WIP to address #421, adding section and subsection numbers throughout the document.
Will require extensive checking of a RTD and PDF rendered version to ensure it is doing what it should and doesn't have any bad side effects.
@Arshitha, @effigies or @sappelhoff: Can you help me make/get access to a RTD and PDF rendered version of this PR?

FYI, content for CSS directly copied and pasted from here:  https://stackoverflow.com/questions/48029165/is-there-a-way-to-make-the-headings-sections-and-subsections-numbering-in-markd

@nicholst to get the RTD preview it'd be best if you could make a branch from the main repo instead of from your fork (you should have the rights to do that). Then we can switch on the rendering of that branch from the RTD online panel.

Thanks @sappelhoff... I've created https://github.com/bids-standard/bids-specification/tree/AddSectionNums
Should I close this PR and open an new one based on that branch?

I think you can also change the target of this PR to the new branch and merge it (and yes, then continue work on the branch off this repo)

clicking on the edit button and then selecting your branch instead of master as the target.

RTD already renders that branch, see: https://bids-specification.readthedocs.io/en/addsectionnums/

I would like to merge this PR into the https://github.com/bids-standard/bids-specification/tree/AddSectionNums branch so I can create a new PR based on that (renderable) branch.  Can an admin please allow this? @sappelhoff

I just tried to do that for you @nicholst but I saw that you already pushed the same changes to bids-standard:AddSectionNums that you have in nicholst:AddSectionNums, so we can just close this PR and you can continue working in a new PR.
Sorry for the confusion :-)


message_reaction: add timestamp

@GAMELASTER do you have any problem with this?

@Schmavery oh sorry, no no, I just sad because I again forgot something 😞

Ahh don't worry about it.  I think we have pretty inconsistent handling of timestamps in the other events too.


[WIP] add physical server custom buttons

I was asked to see how much would be required to add physical server custom buttons and I'm hardly the SME for cbs so I need this 🍒 reviewed as I don't have creds for lenovo and thus cannot test.
@miq-bot assign @tinaafitz
@miq-bot add_label enhancement
it's for https://bugzilla.redhat.com/show_bug.cgi?id=1858171
and yeah, i know i was asked to scope it, not do it, whatever.

MH pointed out that this requires a change to app/helpers/application_helper/toolbar/mixins/custom_button_toolbar_mixin.rb

Just kidding, Tina said to close this.


Add 'Backlog' app

Repo url: https://github.com/czytelny/backlog
Screenshot:
Best regards

Nice. Do you want to add your screenshot to the app metadata?
See https://github.com/electron/electron-apps/blob/master/contributing.md#adding-your-app

@zeke thanks, screenshot added. hopefully I did it properly:)

👍


It seems that v1 version performance is much better than v2, what is the reason for this?

Please fill out the issue template and provide a comparison repro or I will close this issue.


fix: updated intro video on the docs main page

Description
Provide a concise summary of the changes made in this pull request
Pull request type
Check the appropriate box:

[ ] Review Fixes
[X] Documentation Overhaul
[X] Feature/Story

Link one or more Engineering Tickets

https://github.com/appsmithorg/appsmith-docs/issues/2571

[ ] A-Force
[] Error in documentation
[ ] Maintenance

Documentation tickets
Link to one or more documentation tickets:
Checklist
From the below options, select the ones that are applicable:

[ ] Checked for Grammarly suggestions.
[ ] Adhered to the writing checklist.
[ ] Adhered to the media checklist.
[ ] Verified and updated cross-references or added redirect rules.
[ ] Tested the redirect rules on deploy preview.
[x] Validated the modifications made to the content on the deploy preview.
[ ] Validated the CSS modifications on different screen sizes.

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

@KirankumarAmbati - Thank you for raising this PR and your efforts in embedding the video! Upon reviewing the video, we noticed that the video requires some modifications to align with our doc and design guidelines. Our internal team will take this forward from here, so we won’t merge this PR. However, your contribution is greatly appreciated.


Bookmarks and History hover

Description of the new feature / enhancement
Hover over the history/bookmarked manga/chapter in home screen and you can see the name of manga, chapter, pages, date.

Can you add the colon for both of them too as you did for the reader for the upcoming update. Thank you.

Fixed. Will be released in next update.


How to sign with RSA PSS mode

Hello, I want to use RSA PSS mode to sign,but the command I cannot find, could you give me some advises or examples? Thank you very much.

It's unclear from your question whether you are asking about the OpenSSL command line application, or the libcrypto library. Are you looking for API examples or command line examples?

marking as inactive, will be closed at the end of 3.4 dev barring further input


BUG: Workaround for segfault in itkStatisticsAlgorithmTest (v140)

Fixes the issue for VS2015 but I don't have _MSC_FULL_VER 191025017 #1287

@Nekto89 thanks! The Linux Python test failures are unrelated.


Improvements to README.md

The readme section of the project is very basic. A lot of improvements can be done.

Hey
Can I work on this?

Hey
Can I work on this?

Sure. I am assigning you the issue.

Sure. I am assigning you the issue.

Hey @kanishkaa24, any updates?

Hey @kanishkaa24, any updates?

Hey @kanishkaa24, any updates?

I'll make the PR tomorrow. It's under progress.

Hey @kanishkaa24, any updates?

I'll make the PR tomorrow. It's under progress.

Do you have any logo ?

Do you have any logo ?

You can find it here

You can find it here

Can you tell me the procedure to run the code locally ?

Can you tell me the procedure to run the code locally ?

You will need Android Studio on your system.

You will need Android Studio on your system.

I am planning to add it in the readme section so that the contributors can work accordingly.

I am planning to add it in the readme section so that the contributors can work accordingly.

Ok, draft it out. I will add my inputs to your work.

Ok, draft it out. I will add my inputs to your work.

I don't have any idea about this. Could you give me some pointers?

I don't have any idea about this. Could you give me some pointers?

Pointers as in?

Pointers as in?

As in how to install and run code locally

As in how to install and run code locally


Updated and more complete version of the zlib shim

There is a new version of the browserify-zlib shim to zlib available at https://github.com/ipfs/browserify-zlib, it fixes some bugs and more importantly, it adds support for dictionaries.
It seems that not a lot of people use that feature, but it is essential for libp2p-spdy in the browser, which is part of js-libp2p and js-ipfs. Right now we are replacing the shim in our build tool, but it would improve the experience a lot if everyone using it could just rely on webpack to use the correct version of the shim.
We've a open PR -- devongovett/browserify-zlib#18 -- to the bundled browserify-zlib in browserify, but 8 months have passed since it was open, the author and maintainer seems to be lacking the time to review. What else an we do to move this forward?
Thank you :)
There is also an issue open on browserify for this: https://github.com/substack/node-browserify/issues/1672 which suffers from the same problem.
cc @diasdavid

Yes please!

Merging this would be hugely beneficial for the https://github.com/ipfs/js-ipfs project.
We're currently working towards supporting https://github.com/facebookincubator/create-react-app which doesn't support custom webpack configs. As js-ipfs in itself uses a fork of browserify-zlib, we need to include it in our build tools (https://github.com/dignifiedquire/aegir/blob/master/config/webpack.js#L35). As such, we're not able to have create-react-app compile code that uses js-ipfs, because we can't include the fork :/ I hope this makes sense, but let me know if clarification is needed.
If there's anything I (and we) can do to get closer to merging this, please let us know.

@haadcode for us it's important to know if there are breaking changes in the new library compared to the curret one and if so, which one. Also a PR implementing this would be appreciated.

@SpaceK33z thank you for the response. There should not be any breaking changes, only bug fixes and feature additions. I looked through the changelogs again and there where no breaking changes listed for zlib in v4, v5 or v6
If there is interest in shipping this I would publish the fork to npm on a new name and open a PR here. That change would only consist in changing the name of the zlib dependency and its version.

@dignifiedquire go ahead and publish it to npm, name suggestion browserify-zlib-complete :)

@dignifiedquire okay, without breaking changes we can publish this as a minor version. Yes publishing it on npm and opening a PR would be a good idea. Note that I don't maintain this repo myself, but I'll try to get some others on it after your PR.

Published https://www.npmjs.com/package/browserify-zlib-next and created PR #52

beep boop. Can we get an update from the devs on this? Thank you!

Bump ;)
Ran into this, would be nice if it can get merged and released cheers 👍

This is fixed in<EMAIL_ADDRESS>http://npmjs.com/browserify-zlib


the link for the api doc in README returns a 403 error

When I click on the link to the API documentation in the getting started section of README.md wich is http://rust-doc.s3-website-us-east-1.amazonaws.com/tower-web/v0.3.0/tower_web/ ,
I receive a 403 error with message :
403 Forbidden

    Code: AccessDenied
    Message: Access Denied
    RequestId: CF8FE2896C1ECF9C
    HostId: f70IzKWwA0u96mvsRWBotYjhBH+ahXMTSiv2b+Bqg0s+4C70t4C7IX8K+OoKJ12PfGZIlgIKpcI=

See also #104. @carllerche is there a better process that can be put in place?

Sigh! If only doc.rs could process the tower-web docs :(
I also am on a backup laptop. It's going to take me a moment to setup my S3 access again.

I uploaded the docs: http://rust-doc.s3-website-us-east-1.amazonaws.com/tower-web/v0.3.0/tower_web/
There probably is a way to hook it up with CI to deploy docs when tagged.

thanks :)


Add scoring feature

This is a big PR to add a feature that translates the rule violations reported by the validator into simple scores that capture the impact of the rule violations on the API.
Any feedback is welcome. The majority of the changes are stable but I'm still adding a couple more pieces of documentation and maybe one or two more tests. That said, it's ready to review.
The commit breakdown is probably not very helpful - I'm going to try and clean it up before merging.

Example output with the new flag:

:tada: This PR is included in version 1.23.2 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.25.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.5.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


Add Yeelight Crystal YLDL01YL

Add Yeelight Crystal YLDL01YL with both downlight and ambilight measurements
Fixes #613

Thanks for contributing!
model.json needs to be directly in model directory.
Also see https://github.com/bramstroker/homeassistant-powercalc/tree/master/custom_components/powercalc/data/lifx/LIFX BR30 Night Vision/

Arh okay - should be fixed now :)

DOH - 2 secs :D

I'm seeing this:
2022-07-22 16:15:31 ERROR (MainThread) [custom_components.powercalc.sensors.power] light.lampe_esstisch_ambilight: Error setting up calculation strategy: ('No lookup file found for mode', <ColorMode.COLOR_TEMP: 'color_temp'>)

@pannal Could you please create an issue (you can refer to this PR there if you want)? I'll not see this comments anymore as this PR is closed.
Probably you have misconfigured as this light has two separate LUT file one for the downlight, one for the ambilight.
You'll need to manually configure this:
sensor:
  - platform: powercalc
    entity_id: light.lampe_esstisch_ambilight
    manufacturer: yeelight
    model: YLDL01YL/ambilight

Like this


Correction for raw affiliation Independent Researcher and Artist

Correction needed for raw affiliation Independent Researcher and Artist
raw_affiliation_name: Independent Researcher and Artist
new_rors:
previous_rors: 00w7etd19
works_examples: W4386057325;W4386057334;W4386057337;W4387506149;W4386057363;W4386764224
contact: 6edbd62de54dabd73fe9c18be89aaacd:5f4fc5c4193cd055a0dcc88e46f05d @ recherche.gouv.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


[gpt-4o]: Real-time voice functionality

Problem Description
As shown in the online videos, GPT-4o supports real-time language conversation and also has emotion recognition. If Next Chat supports this feature, it will be the most advanced among the open-source clients.
Solution Description
There is a separate voice dialog interface
Alternatives Considered
No response
Additional Context
No response

But unfortunately, the gpt-4o API does not currently have its voice function available, and there is no official documentation to consult for configuration.

但遗憾的是，gpt-4o API 目前没有可用的语音功能，也没有官方文档可供查阅进行配置。

Azure has recently been  published a model for real-time speech. Since azure always releases the latter model after openai, openai should release the same model


remove the default cst="None" from colors.get call

This makes the api of pywal incompatible with pywal16 as to get consistent cache filenames I'd have to pass the cst kwarg, pywal does not expose or require this keyword and neither should pywal16, just leave it empty if not provided.
https://github.com/eylles/pywal16/blob/d8fb64d9927afb287c54b852a3530f4b23608f2a/pywal/colors.py#L387-L391
produces when calling pywal.colors.get(         image,         light=(is_file and light_theme),         backend=settings.get("backend", "wal"),         cache_dir=WPG_DIR     ):
'_home_fernando__config_wpg_wallpapers_golden_gai_JPG_9_dark_wal__None_7499119_2.0.0.json'

and produces when calling cache_fname(filepath, backend, False, WPG_DIR):
'_home_fernando__config_wpg_wallpapers_golden_gai_JPG_9_dark_wal___7499119_2.0.0.json'

this breaks wpgtk ability to edit a theme and directly access the cache file consistently without having to check pywal's version.

please confirm that #93 does produce the names expected by wpgtk before i merge and cut a new release.

confirming!

great!


Remove mediaType from search.query state

Fixes

Fixes #835 by @obulat
Description

As the distinction between current mediaType and current searchType was removed, this PR removes mediaType from the search state query property.
It also renames contentType to searchType to improve clarity. searchType refers to what kind of content is being searched, which can include the special ALL_MEDIA type ("All content" in the UI), and mediaType only refers to the actual single media types that are queried by API ("image", "audio").
Testing Instructions

Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[x] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

There is actually an issue with this PR, it causes the "load more" button to appear when there aren't any more results to load: #859
The issue is closed because it's just with this PR but I'm linking it because it includes reproduction instructions.

Thank you, I'll look into this

There is actually an issue with this PR, it causes the "load more" button to appear when there aren't any more results to load: #859
The issue is closed because it's just with this PR but I'm linking it because it includes reproduction instructions.

I've checked now, and this issue is gone.

Thank you so much for noticing all the bugs @sarayourfriend , I would have never found it! It should be fixed now, and I'll add an e2e test for this tomorrow morning.


[Fix] Fix duplicated logging messages

Fix https://github.com/vllm-project/vllm/pull/1494#issuecomment-1787481735

Ah, well I now understand what that setting is for.
Thanks for fixing.


Add test for greater/less than

It can be helpful to have a test for values that are greater or less than
another value, especially when looping over lists and using select() or
selectattr() filters. This patch adds tests that allow a developer to
check if a value is greater or less than another value.
For example, if I have a list of vehicles and I want to find the motorcycles:
vehicles | selectattr('wheels', 'lessthan', 3)

Or, if I want to find large trucks:
vehicles | selectattr('wheels', 'greaterthan', 4)

Tests are included.

I'm in favor of this.

Thanks, @jeffwidman. I couldn't figure out how to make the new tests show up in the documentation, though. Please let me know if I'm missing a step somewhere.

All that should be required is modifying jinja2.tests.TESTS, which is used by the jinjatests directive, which you did. If rebuilding the docs doesn't render the new sections, then try wiping the docs/_build directory to do a clean build.

Got it. Thanks, @davidism.

So the docs are building correctly for you?

@davidism: I figured out the issue. The jinja2 installation in my virtualenv was interfering with the documentation build. I installed the version of jinja from this PR and the docs built fine with the new tests. :)

@davidism Do I need to do anything else on this PR? :)

Thanks, @davidism! :)


run planning_benchmark node failed.

i had run the planning_benchmark node ,but it not run well.
benchmarking standard validator
Done, success rate: 500/500
total duration: 45981 miliseconds
per planning query: 91.962 miliseconds

===================
foot print size:4  foot print points size:100
benchmarking lazy validator
planning_benchmark: /usr/include/eigen3/Eigen/src/Core/DenseCoeffsBase.h:118: Eigen::DenseCoeffsBase<Derived, 0>::CoeffReturnType Eigen::DenseCoeffsBase<Derived, 0>::operator()(Eigen::Index, Eigen::Index) const [with Derived = Eigen::Matrix<float, -1, -1>; Eigen::DenseCoeffsBase<Derived, 0>::CoeffReturnType = const float&; Eigen::Index = long int]: Assertion `row >= 0 && row < rows() && col >= 0 && col < cols()' failed. core dumped.

Then i run it in gdb:
#0  __GI_raise (sig=sig@entry=6) at ../sysdeps/unix/sysv/linux/raise.c:51
#1  0x00007ffff657e921 in __GI_abort () at abort.c:79
#2  0x00007ffff656e48a in __assert_fail_base (fmt=0x7ffff66f5750 "%s%s%s:%u: %s%sAssertion `%s' failed.\n%n", assertion=assertion@entry=0x7ffff6f7b360 "row >= 0 && row < rows() && col >= 0 && col < cols()",
    file=file@entry=0x7ffff6f7b2f8 "/usr/include/eigen3/Eigen/src/Core/DenseCoeffsBase.h", line=line@entry=118,
    function=function@entry=0x7ffff6f7b9a0 <Eigen::DenseCoeffsBase<Eigen::Matrix<float, -1, -1, 0, -1, -1>, 0>::operator()(long, long) const::__PRETTY_FUNCTION__> "Eigen::DenseCoeffsBase<Derived, 0>::CoeffReturnType Eigen::DenseCoeffsBase<Derived, 0>::operator()(Eigen::Index, Eigen::Index) const [with Derived = Eigen::Matrix<float, -1, -1>; Eigen::DenseCoeffsBas"...) at assert.c:92
#3  0x00007ffff656e502 in __GI___assert_fail (assertion=0x7ffff6f7b360 "row >= 0 && row < rows() && col >= 0 && col < cols()", file=0x7ffff6f7b2f8 "/usr/include/eigen3/Eigen/src/Core/DenseCoeffsBase.h",
    line=118,
    function=0x7ffff6f7b9a0 <Eigen::DenseCoeffsBase<Eigen::Matrix<float, -1, -1, 0, -1, -1>, 0>::operator()(long, long) const::__PRETTY_FUNCTION__> "Eigen::DenseCoeffsBase<Derived, 0>::CoeffReturnType Eigen::DenseCoeffsBase<Derived, 0>::operator()(Eigen::Index, Eigen::Index) const [with Derived = Eigen::Matrix<float, -1, -1>; Eigen::DenseCoeffsBas"...) at assert.c:101
#4  0x00007ffff6f61aa5 in Eigen::DenseCoeffsBase<Eigen::Matrix<float, -1, -1, 0, -1, -1>, 0>::operator() (this=0x55555585e618, row=-1, col=390) at /usr/include/eigen3/Eigen/src/Core/DenseCoeffsBase.h:117
#5  0x00007ffff6f609f4 in se2_planning::isInCollision (state=..., footprint=std::vector of length 100, capacity 160000 = {...}, gridMap=..., obstacleLayer="occupancy")
    at /home/zk/workspace/se2_nav_ws/src/se2_navigation/se2_planning/src/GridMapLazyStateValidator.cpp:106
#6  0x00007ffff6f603db in se2_planning::GridMapLazyStateValidator::isStateValid (this=0x5555558344f0, state=...)
    at /home/zk/workspace/se2_nav_ws/src/se2_navigation/se2_planning/src/GridMapLazyStateValidator.cpp:67
#7  0x00007ffff6f6ea3b in se2_planning::OmplReedsSheppPlanner::isStateValid (this=0x7fffffffd840, si=0x5555557850d0, state=0x55555583ee60)
    at /home/zk/workspace/se2_nav_ws/src/se2_navigation/se2_planning/src/OmplReedsSheppPlanner.cpp:78
#8  0x00007ffff6f6bf78 in se2_planning::OmplPlanner::<lambda(const ompl::base::State*)>::operator()(const ompl::base::State *) const (__closure=0x555555784250, state=0x55555583ee60)
    at /home/zk/workspace/se2_nav_ws/src/se2_navigation/se2_planning/src/OmplPlanner.cpp:62
#9  0x00007ffff6f6c904 in std::_Function_handler<bool(const ompl::base::State*), se2_planning::OmplPlanner::initialize()::<lambda(const ompl::base::State*)> >::_M_invoke(const std::_Any_data &, const ompl::base::State *&&) (__functor=..., __args#0=@0x7fffffffd140: 0x55555583ee60) at /usr/include/c++/7/bits/std_function.h:302
#10 0x00007ffff765085c in ?? () from /opt/ros/melodic/lib/libompl.so.15
#11 0x00007ffff75ffbf5 in ompl::base::ReedsSheppMotionValidator::checkMotion(ompl::base::State const*, ompl::base::State const*) const () from /opt/ros/melodic/lib/libompl.so.15
#12 0x00007ffff77bc628 in ompl::geometric::RRTstar::solve(ompl::base::PlannerTerminationCondition const&) () from /opt/ros/melodic/lib/libompl.so.15
#13 0x00007ffff76174b2 in ompl::base::Planner::solve(double) () from /opt/ros/melodic/lib/libompl.so.15
#14 0x00007ffff7820446 in ompl::geometric::SimpleSetup::solve(double) () from /opt/ros/melodic/lib/libompl.so.15
#15 0x00007ffff6f6bde8 in se2_planning::OmplPlanner::plan (this=0x7fffffffd840) at /home/zk/workspace/se2_nav_ws/src/se2_navigation/se2_planning/src/OmplPlanner.cpp:38
#16 0x00007ffff6f6e7d2 in se2_planning::OmplReedsSheppPlanner::plan (this=0x7fffffffd840) at /home/zk/workspace/se2_nav_ws/src/se2_navigation/se2_planning/src/OmplReedsSheppPlanner.cpp:56
#17 0x00005555555635d2 in runPlanner (planner=...) at /home/zk/workspace/se2_nav_ws/src/se2_navigation/se2_planning/src/planning_benchmark.cpp:45
#18 0x0000555555563a0a in runLazyValidator (planner=..., gridMap=...) at /home/zk/workspace/se2_nav_ws/src/se2_navigation/se2_planning/src/planning_benchmark.cpp:71
#19 0x000055555556409f in main (argc=1, argv=0x7fffffffdaf8) at /home/zk/workspace/se2_nav_ws/src/se2_navigation/se2_planning/src/planning_benchmark.cpp:119

I teied to solve this problem,but it took a long time.Thank you very much for helping me solve this problem.

Hi learnedonroad, thanks for pointing this problem out. I will have a look into this next week and get back to you

thanks a lot

Hi @learnonroad I just ran the planning_benchmark however, I cannot reproduce the issue. Could you push your branch in case you modified something? Or provide me with some instructions on how to reproduce the problem?
Which Eigen version are you using?

Looks like an out-of-bounds matrix read

Can you reproduce it @tomlankhorst ?

I did not make any changes to the code.My software env is:
ubuntu18.04
ros-melodic sudo apt-get install ros-melodic-desktop-full
eigen3  3.3.4:sudo apt-get install libeigen3-dev
gtest sudo apt-get install libgtest-dev
boost
ompl 1.4 sudo apt install ros-melodic-ompl
when I ran the catkin build，there are some warnings
Warnings   << se2_planning:cmake /home/zk/workspace/se2_test/logs/se2_planning/build.cmake.000.log
CMake Warning at /opt/ros/melodic/share/catkin/cmake/catkin_package.cmake:166 (message):
  catkin_package() DEPENDS on 'Eigen3' but neither 'Eigen3_INCLUDE_DIRS' nor
  'Eigen3_LIBRARIES' is defined.
Call Stack (most recent call first):
  /opt/ros/melodic/share/catkin/cmake/catkin_package.cmake:102 (_catkin_package)
  CMakeLists.txt:37 (catkin_package)

then,I changed the se2_planning/CMakeLists.txt line 47-49.
catkin_package(
  INCLUDE_DIRS
    include
    ${EIGEN3_INCLUDE_DIR}
    test
  LIBRARIES
    ${PROJECT_NAME}
  CATKIN_DEPENDS
   ${CATKIN_PACKAGE_DEPENDENCIES}
  DEPENDS
    Eigen3  ##-->> EIGEN3
    Boost
    ompl ##-->>OMPL
)

This warning disappeared.

CMake Warning at /home/zk/workspace/se2_test/src/se2_navigation/se2_planning/CMakeLists.txt:113 (find_package):
  By not providing "Findcmake_clang_tools.cmake" in CMAKE_MODULE_PATH this
  project has asked CMake to find a package configuration file provided by
  "cmake_clang_tools", but CMake did not find one.

  Could not find a package configuration file provided by "cmake_clang_tools"
  with any of the following names:

    cmake_clang_toolsConfig.cmake
    cmake_clang_tools-config.cmake

  Add the installation prefix of "cmake_clang_tools" to CMAKE_PREFIX_PATH or
  set "cmake_clang_tools_DIR" to a directory containing one of the above
  files.  If "cmake_clang_tools" provides a separate development package or
  SDK, be sure it has been installed.

I did not konw how to fix it.

I don't konw if the above warnings will affect my running results.
Maybe I should try to compile Eigen from source?Or can you tell me your operating env?

I can reproduce the error. It'll be fixed by Thu. Don't worry about the warnings for now.

Hi @learnonroad ,
Thank you for your patience. I added this pull request which should address the issue you opened. Could you please verify that your problem does no longer happen?

@jelavice  It runs successfullt,thank you very much.

For the future, you might want to build in release mode since the code runs much faster then. See README in se2_planning.


xADDomainController: Missing Ensure in MSFT_xADDomainController.schema.mof

I'm pretty sure we need to add [read] String Ensure; in MSFT_xADDomainController.schema.mof. Get-TargetResource in MSFT_xADDomainController.psm1 has Ensure in the return value, which is not specified in the schema file.
This seems to have fixed it locally. Let me know if this sounds right and I will submit a PR.

I use the same fix locally, seems right from my experience.
If not fixed then Get-dsccompliance breaks, with error events 4097 and 4252 in operational log

This should be fixed at the same time as #67

This issue still isn't fixed, I ran into when resolving a different issue relating to the xActiveDirectory module. It is present in version Releasing version <IP_ADDRESS> and hasn't been updated on GitHub as of yet. Let's see if I can figure out this pull request thing. :)

That's the attitude! It's on my list but was waiting to Mr Greene to resolve his merge conflicts.
I see no reason not to proceed as other changes have since been pushed. If you need any help, just ask!

@iainbrighton I've closed the 2 PRs and submitted an issue asking to port the smaller of the two.  I'm working with Mr Greene to resubmit the other request.

This issue has been resolved in a recent PR.


dlib v19.22 (and master) does not build with gcc 4.8.5 on CentOS 7

Expected Behavior
$ python setup.py build shall succeed even with gcc 4.8.5 since centos 7 is still around and the compiler is not rejected in the CMakeLists.txt .
Current Behavior
$ python setup.py build fails
Compile Issue 1
during CMake run there is a first hint during CUDA test compile:
-- *** CUDA was found but your compiler failed to compile a simple CUDA program so dlib isn't going to use CUDA.
-- *** The output of the failed CUDA test compile is shown below:
-- ***
-- ***   Change Dir: /build-dlib/dlib/build/dlib/cuda_test_build
   ***
   ***   Run Build Command(s):/usr/bin/gmake && [ 50%] Building NVCC (Device) object CMakeFiles/cuda_test.dir/cuda_test_generated_cuda_test.cu.o
   ***   /build-dlib/dlib/dlib/cmake_utils/test_for_cuda/../../cuda/../matrix/../vectorstream/vectorstream.h:31:24: error: invalid use of dependent type 'dlib::vectorstream::vector_streambuf<CharType>::size_type'
   ***                size_type read_pos = 0; // buffer[read_pos] == next byte to read from buffer
   ***                           ^
   ***   CMake Error at cuda_test_generated_cuda_test.cu.o.cmake:276 (message):
   ***     Error generating file
   ***     /build-dlib/dlib/build/dlib/cuda_test_build/CMakeFiles/cuda_test.dir//./cuda_test_generated_cuda_test.cu.o

The  problem which gcc 4.8.5 complains about is initialization of the typedef'd size_type this line 31 in dlib/vectorstream/vectorstream.h.
 size_type read_pos = 0; // buffer[read_pos] == next byte to read from buffer

for which gcc 4.8.5 gives this error:
[...]/dlib/dlib/threads/../binary_search_tree/../vectorstream/vectorstream.h:31:34: error: invalid use of dependent type 'dlib::vectorstream::vector_streambuf<CharType>::size_type'

This can be fixed by not initializing read_pos:
 size_type read_pos; // buffer[read_pos] == next byte to read from buffer

which should to no harm since it is initialized in the constructor as well.
Compile Issue 2
However, when fixing compile issue 1 this issue shows up:
[ 68%] Building CXX object dlib_build/CMakeFiles/dlib.dir/cuda/gpu_data.cpp.o
In file included from /build-dlib/dlib/dlib/filtering/../global_optimization.h:8:0,
                 from /build-dlib/dlib/dlib/filtering/kalman_filter.cpp:7:
/build-dlib/dlib/dlib/filtering/../global_optimization/find_max_global.h:284:47: error: default argument missing for parameter 8 of 'template<class funct, class ... Args> std::pair<long unsigned int, dlib::function_evaluation> dlib::impl::find_max_global(double, dlib::thread_pool&, std::vector<_RealType>&, std::vector<dlib::function_spec>, std::chrono::nanoseconds, dlib::max_function_calls, double, Args&& ...)'
         std::pair<size_t,function_evaluation> find_max_global (
                                               ^
/build-dlib/dlib/dlib/filtering/../global_optimization/find_max_global.h:303:47: error: default argument missing for parameter 7 of 'template<class funct, class ... Args> std::pair<long unsigned int, dlib::function_evaluation> dlib::impl::find_max_global(double, dlib::thread_pool&, std::vector<_RealType>&, std::vector<dlib::function_spec>, std::chrono::nanoseconds, double, Args&& ...)'
         std::pair<size_t,function_evaluation> find_max_global (
                                               ^
In file included from /build-dlib/dlib/dlib/svm/../global_optimization.h:8:0,
                 from /build-dlib/dlib/dlib/svm/auto.cpp:7:
/build-dlib/dlib/dlib/svm/../global_optimization/find_max_global.h:284:47: error: default argument missing for parameter 8 of 'template<class funct, class ... Args> std::pair<long unsigned int, dlib::function_evaluation> dlib::impl::find_max_global(double, dlib::thread_pool&, std::vector<_RealType>&, std::vector<dlib::function_spec>, std::chrono::nanoseconds, dlib::max_function_calls, double, Args&& ...)'
         std::pair<size_t,function_evaluation> find_max_global (
                                               ^
/build-dlib/dlib/dlib/svm/../global_optimization/find_max_global.h:303:47: error: default argument missing for parameter 7 of 'template<class funct, class ... Args> std::pair<long unsigned int, dlib::function_evaluation> dlib::impl::find_max_global(double, dlib::thread_pool&, std::vector<_RealType>&, std::vector<dlib::function_spec>, std::chrono::nanoseconds, double, Args&& ...)'
         std::pair<size_t,function_evaluation> find_max_global (
                                               ^

Seems that the "old" gcc cannot cope with variadic/Parameter pack? function declaration (probably because there is a default value set before for solver_epsilon (line 291):
        std::pair<size_t,function_evaluation> find_max_global (
    285             double ymult,
    286             thread_pool& tp,
    287             std::vector<funct>& functions,
    288             std::vector<function_spec> specs,
    289             const std::chrono::nanoseconds max_runtime,
    290             const max_function_calls num,
    291             double solver_epsilon = 0,
    292             Args&& ...args
    293         )

Unfortunately I could not find a solution for that yet...
Steps to Reproduce
run
`$ python setup.py build`

with gcc 4.8.5

Version: v19.22 and current master
Where did you get dlib: GitHub repository
Platform: CentOS 7 base on nvidia/cuda docker image: nvidia/cuda:10.0-cudnn7-devel-centos7 + dev tools
Compiler: gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-44)

Yeah, dlib v19.22 dropped support for gcc 4.8, try with a previous version.

That's a pity. There are still a lot of CentOS 7 installations out there and it will get maintenance update until 2024. However, if this is the decision we have to live with it. Yet it would be good to write this to the README.md or/and catch it by the CMake run.
Thx anyway. Keep up the great work.

It's not like you can't use dlib anymore, just install v19.21 :)

Heh, yeah, I try and go wack by with compiler support.  But it's 2021 and I've only moved dlib to C++11.  And even then I've tried to avoid breaking compilers that have only partial C++11 support.   But gcc 4.8.5 is just too non-C++11 compliant at this point to be tenable.
And like @arrufat pointed out.  Just don't upgrade to the newest dlib version and you are good.

Thx for your feedback - closing the issue.


feat: Startup logic

@six7 I streamlined the startup logic as I had mentioned before. Now the plugin data will be pre-fetched in one go and sent to the front-end where it will be handled in a single step-by-step process reducing the amount of round-trips and making the plugin logic more clear to reduce the possibility for errors.
I did alter the loading behavior of the plugin slightly where the initial pulling of the tokens will now also happen during the loading screen, but the loading screen has also been updated with more clear information for the user (clearly stating the different steps; such as verifying the license, getting the LD flags etc, ...).
The process now also ensures the LD flags are ready before initializing any other logic in the plugin ensuring the right features are available to the user at the right time.
As a TODO to myself - I need to adjust the tests where required

https://user-images.githubusercontent.com/5265324/180692517-f7a59109-eedd-4ff7-a068-65ee9704abc8.mp4

This is awesome, I love the various loading states! I tried with a couple different scenarios but will do some more testing and then look at the code. Thank you for this! 🎉

@six7 updated all the tests etc..

One thing I noticed: If there's a credentials error during startup the plugin stays in Fetching credentials loading state forever. This happened with ADO sync.

@six7 the error occurs if the local api credentials are invalid or something?

@six7 issue with the loading state should be fixed

Can you resolve merge conflicts? After that let's merge this in.


fortSearch respond with experience_awarded = 0

Received status = 1 in response of fortSearch, but experience_awarded = 0 most of the time.

Are you getting items? And are all pokemon you try to catch fleeing? It sounds like a soft ban.

Usually experience_awarded equals to zero with no items dropped.
But sometimes i got 50 or 100 experience with items.
I can't understand why the most of the time i receive 0 experience.

It is a soft ban, when you travel too fast it prevents you from spinning and all pokemon flee for about an hour. The 100 exp is when you spin 10 stops quickly after each other, obviously without being soft banned.

I have a 15 seconds delay between teleporting. Is it too small? Should I use another tactic instead of teleporting?

Increased delay value to 20 seconds, looks like it's a solution.

You should build in set meter/second constant and make your code move there at that rate, I wouldn't go much higher then 30/mps. Either way this is a implementation issue, we try to keep issues here focused on issues or suggestions for pogobuf, so I'll close this now. Good luck.


Add enabled prop

This pull request makes it possible to disable the drowpdown list

This functionality already exists,
dropdownShown prop does the same

@danish1658 why this has been discarded? Currently there is no way to disable the select completely, disabling the "touchability"
I think it is good option to have


Add support to portuguese from Brazil

The agent have support for portuguese (pt-pt) subtitles but not for portuguese from Brazil (pt-br).
This support would be nice since the dialects are very different.

You'll be better off with https://github.com/pannal/Sub-Zero.bundle that you can Add from within Plex Channels as Subliminal has been frozen for a long while now.


[improve][build] Create images before build python wheels

Motivation
Currently in the release process of the python client, there is no indication that the related images need to be created before building python wheels. This may cause the release manager to build the wheel without using the latest images.
Modifications

Call create-images.sh in the build-wheels.sh
Add option --skip-create-images to the build-wheels.sh to skip the image creation. The default is not skipped.

Documentation
Check the box below or label this PR directly.
Need to update docs?

[ ] doc-required
(Your PR needs to update docs and you will update later)

[x] doc-not-needed
(Please explain why)

[ ] doc
(Your PR contains doc changes)

[ ] doc-complete
(Docs have been already added)

@RobertIndie In theory the images should already be available in Docker hub and it takes a lot of time to rebuild them. Is there any particular image that is missing?

@merlimat IMO, maintaining the images in DockerHub manually is not a good way to do that, especially because only a few members have the permission. If the release manager forgot to run docker update, an outdated build image could be used.
The basic build image usually installed some dependencies by downloading and building. It should not take much time. But I found it could be speeded up significantly by reducing the output. For example, replace tar xvfz with tar xvz.

@RobertIndie In theory the images should already be available in Docker hub and it takes a lot of time to rebuild them. Is there any particular image that is missing?

For example, the version of OpenSSL is wrong in the pulsar python client 2.10.1. You can see more detail here: https://github.com/apache/pulsar/issues/17097#issuecomment-1236873140
I think this is because the latest image was not used in the release 2.10.1 process.

IMO, maintaining the images in DockerHub manually is not a good way to do that, especially because only a few members have the permission. If the release manager forgot to run docker update, an outdated build image could be used.

@BewareMyPower If we add a docker pulsar $IMAGE it will force to use the version that exists on DockerHub

@merlimat Did you mean docker pull? What I concerned more about is that the image might not be built with the latest Dockerfile because it's uploaded manually. At the moment when uploading images cannot be executed automatically, we'd better support building the base image.

Creating the images takes a very long time. We should have the images built once, available on Docker hub and cached as needed.
By doing docker pull $IMAGE before using it, we're making sure that when the image is manually updated, the builds will start using it.

In theory the images should already be available in Docker hub and it takes a lot of time to rebuild them.

I agree now. I've thought the create-images.sh is similar to building base images for RPM and DEB packages. I took a look just now, this script will create nearly 10 images, which takes too much time.
pulsar-build:manylinux2014-cp37-cp37m-x86_64
pulsar-build:manylinux2014-cp38-cp38-x86_64
pulsar-build:manylinux2014-cp39-cp39-x86_64
pulsar-build:manylinux2014-cp310-cp310-x86_64
pulsar-build:manylinux2014-cp37-cp37m-aarch64
pulsar-build:manylinux2014-cp38-cp38-aarch64
pulsar-build:manylinux2014-cp39-cp39-aarch64
pulsar-build:manylinux2014-cp310-cp310-aarch64
pulsar-build:manylinux_musl-cp37-cp37m-aarch64
pulsar-build:manylinux_musl-cp38-cp38-aarch64
pulsar-build:manylinux_musl-cp39-cp39-aarch64
pulsar-build:manylinux_musl-cp310-cp310-aarch64
pulsar-build:manylinux_musl-cp37-cp37m-x86_64
pulsar-build:manylinux_musl-cp38-cp38-x86_64
pulsar-build:manylinux_musl-cp39-cp39-x86_64
pulsar-build:manylinux_musl-cp310-cp310-x86_64

By doing docker pull $IMAGE before using it, we're making sure that when the image is manually updated, the builds will start using it.

My point is not about whether the local image is latest. I mean the process is complicated and needs some manual operations. We must document that once any of the following files are changed, we must notify a maintainer of apachepulsar DockerHub account to upload the newest image.

python-versions.sh, which specifies the supported Python versions and the associated base images
manylinux2014/Dockerfile, the base image of Python 3.7 to 3.10 for x86_64 and aarch64 architectures
manylinux_musl/Dockerfile, the base image of Python 3.7 to 3.10 for Alpine compatible wheels

If we want to change these Dockerfiles, for example, the xvfz option for tar command takes much more time than xvz for the extra output. Even this minor optimization requires a complicated update steps.
In addition, we should not use such a tag like manylinux_musl-cp310-cp310-x86_64. It should be a part of image name, the tag should be a version.

In conclusion, I agree that we should not add an option to create base images for building Python wheels currently. But we should make the process clear in case when the base image changed.
/cc @merlimat @RobertIndie

I realized the key point suddenly. https://github.com/apache/pulsar/pull/17538 upgrades the OpenSSL version to 1.1.1n for some Dockerfiles, but even it has been merged into the master, it's hard to know:

whether the latest images in DockerHub include this change? If yes, how to keep the compatibility with the previous image that might have been overwritten.
are these images related to Python clients?

See the related issue: https://github.com/apache/pulsar/issues/17097
/cc @merlimat @liliang950210

Closed as the development of the Python client has been permanently moved to https://github.com/apache/pulsar-client-python.


Seperate Emote Window

What should be added?
Please make the Emote Window like the one in Twitch Chat or atleast give the option to remove the current Emote Window pop up feature or change its design.
Why should it be added?
Everytime i click on the Emote Button , the pop up window with the emotes, appears on the wrong screen and/or wrong position. plus having always to click the X to close it, is annoying, if you are used to the original twitch chat function , which is just one click in chat and the emote window disappears and it always opens where the chat window is. would be awesome to have it both ways :) keep up the great work

So you'd like a setting that would make the emote picker window close when you click an emote?

Hi,
nah ^^ at the moment the emote window and the chat window are seperate, i like to have it like in the screenshot. you can see that  no matter if i click twichs emote button or bttv's, it always seem to be a feature of the chat and not a seperate window. i personally like it better as shown on the screenshots, since it lets me simple click in the anywhere in chat to cancel the emote window, iirc to cancel twitch's emote window by clicking anywhere in chat (where it displays everybodys messages), and for bttv you can just click anywhere in the window where you have to type messages to chat. i personally prefer bttv's settings since its easier to cancel.
hope that makes more sense 😄 sorry for my bad english thou and thanks again for the work you guys put into it
Cheers Bake

Von: Mm2PL<EMAIL_ADDRESS>Gesendet: Montag, 14. Dezember 2020 06:17
An: Chatterino/chatterino2<EMAIL_ADDRESS>Cc: BakeFlowers<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [Chatterino/chatterino2] Seperate Emote Window (#2280)
So you'd like a setting that would make the emote picker window close when you click an emote?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Chatterino/chatterino2/issues/2280#issuecomment-744469512, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASDYRDNV4EWZOQPTV6V3EMLSUYM63ANCNFSM4U2ZYEEA.

like in the screenshot

as shown on the screenshots

Am I missing something or did you mean to attach some screenshots, but you forgot in to include them? I am a little confused with your request since I never used the web extensions for chat, but maybe someone else will get it.

You would like the emote window to be inline inside that chat window, just like the emote suggestions if you type :thu?

Hi
sorry for the late reply, all i want is that its one window. no need to have two seperate windows imo, like i said, if you go to e.g. twitch and use the bttv extension and click on the lil smiley next to the chat button you ll see what i  mean. so the emote window is still poping up but its bound to the main chat window
i ll try to provide more informations, to stressful atm ^^
merry xmas/holydays and a good slide in the new year

Von: pajlada<EMAIL_ADDRESS>Gesendet: Samstag, 19. Dezember 2020 03:00
An: Chatterino/chatterino2<EMAIL_ADDRESS>Cc: BakeFlowers<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [Chatterino/chatterino2] Seperate Emote Window (#2280)
You would like the emote window to be inline inside that chat window, just like the emote suggestions if you type :thu?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Chatterino/chatterino2/issues/2280#issuecomment-748457944, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASDYRDKJOFZU5R2GOSD3T73SVSBUJANCNFSM4U2ZYEEA.

Hi
sorry for the late reply, its been busy around this time of the year.... ^^
I might have forgot to send you  the screenshots
its prolly hard to see what i mean, when you never used the bettertwitchtv extension.
explained it to your friend already.
Gonna provide more infos for sure, just need couple days...
merry xmas/holydays and happy new year

Von: Paweł<EMAIL_ADDRESS>Gesendet: Freitag, 18. Dezember 2020 05:26
An: Chatterino/chatterino2<EMAIL_ADDRESS>Cc: BakeFlowers<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [Chatterino/chatterino2] Seperate Emote Window (#2280)
like in the screenshot
as shown on the screenshots
Am I missing something or did you mean to attach some screenshots, but you forgot in to include them? I am a little confused with your request since I never used the web extensions for chat, but maybe someone else will get it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Chatterino/chatterino2/issues/2280#issuecomment-748083239, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASDYRDI34XSBSETXPCJWOFLSVNJ7TANCNFSM4U2ZYEEA.


Kotlin 2.1.0

Closes #403.

> Task :mpp:transformCommonMainDependenciesMetadata
e: KLIB resolver: Could not find "/home/runner/work/Poko/Poko/sample/mpp/build/kotlinTransformedMetadataLibraries/commonMain/org.jetbrains.kotlin-kotlin-stdlib-2.1.0-Beta2-commonMain-ITmP6A.klib" in [/home/runner/.local/share/kotlin/daemon]

> Task :mpp:compileCommonMainKotlinMetadata FAILED
w: KLIB resolver: Skipping '/home/runner/work/Poko/Poko/sample/mpp/build/kotlinTransformedMetadataLibraries/commonMain/org.jetbrains.kotlin-kotlin-stdlib-2.1.0-Beta2-commonMain-ITmP6A.klib'. Incompatible ABI version. The current default is '1.8.0', found '1.201.0'. The library was produced by '2.1.0-dev-7621' compiler.

Looking at this locally, that exact filename is present, so the "could not find" error message is strange. But maybe the warning is a hint. Did klib ABI compat break between 2.0.21 and 2.1.0-Beta2?

@JakeWharton found that 2.1.0 updates LLVM to an incompatible version, hence the error.


fix invalid assert dtAssert(npath < m_maxPath)

In the function dtPathCorridor::setCorridor
	dtAssert(npath < m_maxPath);

the assertion failure caused a crash on our server.
After analyzing the source code of this file, it was found that the value assigned to m_npath was used as a size rather than an index, and all the places where it was used would not lead to direct access.
in function dtCrowd::updateMoveRequest
    // Make space for the old path.
    if ((npath-1)+nres > m_maxPathResult)
        nres = m_maxPathResult - (npath-1);

    memmove(res+npath-1, res, sizeof(dtPolyRef)*nres);
    // Copy old path in the beginning.
    memcpy(res, path, sizeof(dtPolyRef)*(npath-1));
    nres += npath-1;

there is a possibility of nres == m_maxPathResult, so this value is normal.
If there is any misunderstanding, please correct me. Thank you.

Thanks!


[sw/silicon_creator] Initialize cpuctrl CSR using OTP

This PR adds the CREATOR_SW_CFG_CPUCTRL OTP item and writes its value to the cpuctrl CSR (after masking with 0x3f to preserve the double_fault_seen sync_exc_seen and bits). This way we can control and test dummy_instr_mask, dummy_instr_en, and data_ind_timing in addition to icache_enable.
Please let me know if I should update the mask to preserve the reset values of the bits other than icache_enable.
Please see this page for more information on the cpuctrl CSR.
This PR also sets the value of this entry to 0x1 in the RMA OTP image. I'll give the resulting bitstream a try before merging this PR.

Thanks @moidx! Rebased, refactored pre-boot checks to mask_rom_pre_boot_check(), added additional checks for the cpuctrl CSR.
Relevant part from the disassembly:
/proc/self/cwd/sw/device/silicon_creator/mask_rom/mask_rom.c:262
      otp_read32(OTP_CTRL_PARAM_CREATOR_SW_CFG_CPUCTRL_OFFSET);
    8bec:       10c00513                li      a0,268
    8bf0:       165030ef                jal     ra,c554 <otp_read32>
/proc/self/cwd/sw/device/silicon_creator/mask_rom/mask_rom.c:263
  CSR_READ(CSR_REG_CPUCTRL, &cpuctrl_csr);
    8bf4:       7c0025f3                csrr    a1,0x7c0
launder32():
/proc/self/cwd/./sw/device/lib/base/hardened.h:243
    8bf8:       862e                    mv      a2,a1
mask_rom_pre_boot_check():
/proc/self/cwd/sw/device/silicon_creator/mask_rom/mask_rom.c:264
  if (launder32(cpuctrl_csr) != cpuctrl_otp) {
    8bfa:       0ca61c63                bne     a2,a0,8cd2 <mask_rom_boot+0x316>
/proc/self/cwd/sw/device/silicon_creator/mask_rom/mask_rom.c:267
  HARDENED_CHECK_EQ(cpuctrl_csr, cpuctrl_otp);
    8bfe:       00a58663                beq     a1,a0,8c0a <mask_rom_boot+0x24e>
    8c02:       0000                    unimp
    8c04:       0000                    unimp
    8c06:       0000                    unimp
    8c08:       0000                    unimp

BTW, the previous initial values for CFI counters were not all 11-bits (at least one of them was 12-bits) contradicting with the comment above the macro.
Also, I'm not sure why there was a inconsistency in formatting. I'll drop the last commit if this was due to a clang-format version mismatch.

Also, I'm not sure why there was a inconsistency in formatting. I'll drop the last commit if this was due to a clang-format version mismatch.

I think I know why: the X-macro that defines the initial values for the CFI counters ends with a \ in master eating the // clang-format on in the next line.

I tried the CW310 bitstream with the mask ROM and CREATOR_SW_CFG_CPUCTRL OTP item set to 0x1 but I didn't receive anything over UART. Commenting out the CSR_WRITE() (or writing 0) resolves the issue. I reverted the changes in DEV and RMA OTP hjson files, i.e. CREATOR_SW_CFG_CPUCTRL is now 0. I'll try the new bitstream to make sure that it works as expected.

@alphan do you want to push the changes that failed into this PR?
Most likely the failure will be seen in simulation as well, as we can debug it (unless you were seeing the failure only on fpga?)

@alphan do you want to push the changes that failed into this PR? Most likely the failure will be seen in simulation as well, as we can debug it (unless you were seeing the failure only on fpga?)

@tjaychen this was most likely a false alarm (reading from the OTP before sec_mmio_init()). I'll push an update but things seem to be OK on CW310 locally.

bazel run //sw/host/opentitantool -- bootstrap --protocol eeprom `pwd`/bazel-bin/sw/device/silicon_creator/rom_ext/virtual_addr_fpga_cw310.test_key_0.signed.bin

MaskROM:5b2c4370
MaskROM:5b2c4370
starting rom_ext
BFV:0142500d
LCV:2739ce73

thanks @alphan i'll update and take a look at the waves.
I'm a little surprised the flash didn't fall over...

o wait, i realized just now we did not apply this change to test_rom.
Should we do the same there?  Should we do the same there? Otherwise most of our DV / fpga won't get any coverage.

I think it's a good idea, but really up to you. @timothytrippel you can take a look at this change but I'm not sure you'd care about the OTP stuff. Basically all you need is to write 1 to the cpuctrl CSR.

thanks @alphan i'll update and take a look at the waves. I'm a little surprised the flash didn't fall over...

We don't have a ton of coverage for this yet but works on my machine™.

@timothytrippel if you're too busy to look at this, i can have a look later today as well.

Yeah I think it's a good idea to write the cpuctrl register in test ROM too. if we wanted to do an OTP copy tho we would have to update the OTP image that used in testing to include the relevant bits. @alphan when you say we just need to "write 1" to the cpuctrl register, are you saying we would just enable the icache in test ROM? and not configure any of the other bits?

The benefit of making it the same OTP copy as mask ROM is we get closer feature parity, but we need a better mechanism for generating various OTP images for tests (which is on the ever growing TODO list hehe)

Yeah I agree, especially with the modular source code bit if we want feature parity. So for the time being maybe we just turn on icache in test ROM and call it a day?

Yeah I agree, especially with the modular source code bit if we want feature parity. So for the time being maybe we just turn on icache in test ROM and call it a day?

Sure. We can revisit this discussion when we start touching other bits. @tjaychen WDYT?

we could make it a configuration parameter like the kJitterEnabled, and let the test bench backdoor overwrite the symbol. but this will only work on DV platform obviously. not FPGA.

yeah that sounds good to me!


Using django-netjsongraph with Networkx

Can I use networkx with django-netjsongraph to display graphs in the browser?

@ImanoWilliams that's the purpose of this project, but your question is very broad, this app is focused on displaying network graph data which is retrieved from mesh routing deamons or any other software capable of returning NetJSON output


[FEATURE] Use Socket.IO instead of fetch

Is your feature request related to a problem? Please describe.
The YTMD-App should send the information instead of requesting it all xxx ms.
Describe the solution you'd like
Use SocketIO instead of fetch.
Describe alternatives you've considered
Example script:
<!-- Required to include the Socket.IO library -->
<script src="https://cdnjs.cloudflare.com/ajax/libs/socket.io/2.3.0/socket.io.js"></script>
<script>
    const server = "http://localhost:9863";
    const socket = io(server);

    socket.on("connect", () => {
        console.debug('Connected')
    });

    socket.on("disconnect", () => {
        console.debug('Disconnected')
    });

    socket.on("tick", (data) => {
        console.debug(data);
    });
</script>

/cib


Implement executor::Enter

This implements executor::Enter as described by the Tokio reform RFC. This helps prevent accidentally blocking an executor thread.
This does not update CpuPool to use enter as the strategy by which to do this is not clear. Specifically, calling executor::enter from CpuPool threads is easy, but there is no way to expose the &Enter handle to the user (this also probably shouldn't be exposed directly).

This does not update CpuPool to use enter as the strategy by which to do this is not clear. Specifically, calling executor::enter from CpuPool threads is easy, but there is no way to expose the &Enter handle to the user (this also probably shouldn't be exposed directly).

So the intent, in my mind, was that &Enter should only be exposed during the initialization phase of an executor (e.g., it may allow the client to register an initialization hook). I believe some parts of the RFC depend on you not being able to get your hands on an &Enter arbitrarily.

@aturon either way, updating CpuPool doesn't need to block this PR. It can be tracked in the meta issue. So, let me know if you have any other change requests. If not, plz give the +1.

Approved.


Proof of concept for an npm managed setup on the hello-world exercise

This is a first stab at switching the tests to using a local package.json for each exercise. This enables the user to use a copy of elm/elm-test specific to that exercise. This should give us more flexibility around versioning and is just more hygienic overall.
Background:

Upgrading to Elm 0.18

Highlights:

AFAICT the contents of SETUP.md will be appended to the README.md that is fetched with each exercise. This should provide up to date install/test instructions even if they've previously had old versions of previous exercises.
I'm not sure I want to muddy the waters with mentioning yarn, but I'm tempted to leave a yarn.lock file for anyone who cares to use it. yarn install and yarn test seem to work just fine.
I renamed the test and example files to simple Tests.elm and Example.elm. This makes it easier to ditch the wildcard in the test command (better for windows) and enables syntax highlighting in the example file.

Known todos:

[ ] Get some feedback
[ ] Test on windows (I think it'll work)
[x] Test on linux (I'm pretty sure it'll work)
[ ] Migrate the rest of the exercises
[x] Update the central elm docs/build.sh

CC: @fpapado @chadfennell @ryantriangles @rebelwarrior

I like the idea of supporting multiple versions in order to make it easier to upgrade exercises incrementally when a new release occurs.

Tests.elm and Example.elm

Yes, for the reasons you mentioned. I mistakenly submitted my first kata in this format, seems like a natural default.

[x] Test on linux

Works on Ubuntu 16.04.

Ok, looks like the central build part is working. I had to back off the example rename a bit in lieu of coming up with a more clever way to do the detection in build.sh, but it's still an improvement.

Should be good to go. I'll let it sit overnight in case anyone has an last minute thoughts/objections (and until I have a chance to do a smoke test in windows).

Windows seems 👍

Hey @tgecho, great work! Didn't get to reply on the earlier gb issue, I agree that this seems the best approach.


Scaffolding for integration test

Coverage decreased (-0.02%) to 72.287% when pulling 02b36330f7303f44db4c041e0d5042c702f7d0e4 on asifdxtreme:integration-test into<PHONE_NUMBER>d2edb249fdaf965774b3ec10a33431 on ServiceComb:master.

Codecov Report

Merging #25 into master will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #25      +/-   ##
==========================================
+ Coverage   68.29%   68.32%   +0.02%
==========================================
  Files          14       14
  Lines        3391     3391
==========================================
+ Hits         2316     2317       +1
  Misses        852      852
+ Partials      223      222       -1

Impacted Files
Coverage Δ

common/cron/cron.go
53.97% <0%> (+0.56%)
:arrow_up:

Continue to review full report at Codecov.

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

Coverage increased (+0.07%) to 72.378% when pulling 5d1e6fc7dee4f0df42ce7e4024bb7825d94fb7b2 on asifdxtreme:integration-test into<PHONE_NUMBER>d2edb249fdaf965774b3ec10a33431 on ServiceComb:master.

Coverage increased (+0.1%) to 72.445% when pulling 8e1579f687fa439a47ac2fe86d41de5ddfe25735 on asifdxtreme:integration-test into<PHONE_NUMBER>d2edb249fdaf965774b3ec10a33431 on ServiceComb:master.

Coverage decreased (-0.002%) to 72.31% when pulling 29be11ec733e1df5d08ad70d43b4e8a8f5a4ef30 on asifdxtreme:integration-test into<PHONE_NUMBER>d2edb249fdaf965774b3ec10a33431 on ServiceComb:master.

Coverage increased (+0.02%) to 72.337% when pulling a13c3b86a1d505ce50bc993b1258240db7a6c084 on asifdxtreme:integration-test into<PHONE_NUMBER>d2edb249fdaf965774b3ec10a33431 on ServiceComb:master.

Coverage decreased (-0.02%) to 72.287% when pulling bf97be1127af69cca43da6fa9f83bc79cbc4f420 on asifdxtreme:integration-test into<PHONE_NUMBER>d2edb249fdaf965774b3ec10a33431 on ServiceComb:master.

Coverage decreased (-0.02%) to 72.287% when pulling d87c961da1c2be59027571fd78fe35a6198b7943 on asifdxtreme:integration-test into<PHONE_NUMBER>d2edb249fdaf965774b3ec10a33431 on ServiceComb:master.


[Feature] Add a way to use in private or other way to stop single-signon

Description
I think i would be a great idea to add a way to make the compiled app use an in private window or another way to avoid single-signon
Detail
When on a domain joined Windows client where single-signon is setup on the domain between the on-prem AD and Azure Ad with federation browsers logon to Azure and Office 365 services with single-signon. This is a great feature for end users, it is, however, a problem for admins who have to log in to portals with service accounts and have to log in to different tenants. Usually this can be avoided using the in-private feature in browsers. This feature cannot control the logging into different tenants, natifier can
Steps to reproduce issue
-> On a domain joined Windows PC with federation on the domain try to log to portal.azure.com with a different user than the current domain user

Are you nativefying a public website?
--> Yes, https://portal.azure.com
Feature request? Have you looked at nativefier --help to see if an existing option could fit your needs?
--> Nothing seems to deal with that in help
Full nativefier command used to build your app: nativefier "https://portal.azure.com" --name "Azure" --icon "azure-icon-250x250.png" --user-agent "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/52.0.2743.116 Safari/537.36 Edge/15.15063"
Version of Nativefier (run nativefier --version): v7.4.1
Version of node.js (run node --version): v6.11.4
OS: Windows 10 Enterprise (Build 15063.632

Hi @saadanerdetbare and thanks for the feedback. I don't understand, can you re-formulate the issue as narrowly as possible and provide a scenario demonstrating expected vs. actual behavior?


Unable to run tests using Darwin arm64 (Apple M1)

Currently we cannot use setup-env.
Problem:
Architectural limitations while creating the environment test from Envtest lib.
setup-envtest list --os darwin--arch arm64
This option is not available on env-test options list, not allowing to create the environment needed to run Suit tests.
Possible solutions:
Use external cluster options, for that we can enable "UseExistingCluster" option on "Envtest.environment".

For anyone with this issue:
You can pass USE_EXISTING_CLUSTER environment variable to control the behavior of ENVTEST.


Using multiple classes for tag shortcut

Currently only one class can be processed.
d#myId.foo -›

Approach for passing multiple classes
d#myId.foo.bar -›

Hi, Samuel
I have just updated the new version of BracketstoIX.
The tag command supports multiple classes.


[NEW] Swag Opportunity  Bagisto🎉

Name Of Organization
Bagisto
Description
For every 4 PR that gets accepted, you can adopt Baggy The Elephant along with cool swags
Swag Items
Pen, Stickers, stuffed toy
Link to proof
Link to the place where you saw this
https://webkul.com/meetups/hacktober-fest-bagisto-2022/

I can't open a new PR until my last one is approved

I can't open a new PR until my last one is approved
Bagisto is already  added on the live website


implement something like PYFORAPATH

we need to improve the way we organize our python mappings.

having to modify the pyfora package (what if it were installed in /usr/local/lib? you could pip install -e, but you might not be in that case) shouldn't be necessary. What if you need some math function mapped which isn't implemented in our library?
we shouldn't have to explicitly register mappers (for example PurePandas.py has a def generateMappings, and this has to be used in DefaultPureImplementationMappings. This is not user friendly.
We shouldn't have to import all of numpy/pandas/etc unless they're actually used by a client.

:+1:


Incorrect url in pxe module

Is the url in line 204 supposed to be "url2"?
https://github.com/hpc/kraken/blob/d8359801117ee8b6e9def71def6161c7cdc25f19/modules/pxe/dhcp.go#L192-L211
Note: this also exists in pipxe module at line 318 in dhcp.go

Resolved by #168


get stuck on macOS when running command

I using<EMAIL_ADDRESS>and it gets stuck when I trying to run serve command
without any output in terminal

I may have found the reason why it stuck
because every time when serve runs, it check for update, if the network environment is not good enough (like inside GFW) 🤐 , it will stuck in the update checking.


Delete master branch

https://github.com/efabless/caravel/tree/master -- It's empty

Lets delete for now, we can recreate it if we want.

@mithro :  I don't know how to remove a branch except locally;  I don't have a master branch in my local cloned copy.

Use the github interface? Or use git push <remote> :master -- Yes that <nothing>, <semicolon>, master -- The value for <remote> is most likely origin but should have a URL like git+ssh://github.com/efabless/caravel.git.

That worked, thanks for the info.


Does not work with node 12

For node 12, fails with:
./node_modules/rollup-plugin-import-assert/dist/import-assert.cjs:89
const assert = { type: node.arguments[0].properties[0]?.value?.properties[0].value.value };
SyntaxError: Unexpected token '.'

with node 14 and 16 it works.
Problem is I have a server with Debian 11.3 that only comes with node 12 and I cannot upgrade

Closing due to lack of response. Also node@12 is a bit old for me to worry about here.

Thanks, we ended up circumventing the problem by just importing js instead of json. That also solved another dev issue of ours, namely that Firefox still does not natively support import assertions. And you are right, node 12 is too old to care. More important in the future might be compatibility with Rollup 3 that got recently released.


Release Google.Cloud.BigQuery.Reservation.V1 version 1.2.0

Changes in this release:
No API surface changes; just dependency updates.

Created release for Google.Cloud.BigQuery.Reservation.V1-1.2.0


Fix linker's warnings about visibility.

Stop the flood of compiler warnings on Mac:
ld: warning: direct access in function 'fmt::v7::appender fmt::v7::detail::write<char, fmt::v7::appender, unsigned long long, 0>(fmt::v7::appender, unsigned long long, fmt::v7::basic_format_specs<char> const&, fmt::v7::detail::locale_ref)' from file '/usr/local/lib/libfmt.a(format.cc.o)' to global weak symbol 'fmt::v7::detail::basic_data<void>::left_padding_shifts' from file '/usr/local/lib/libfolly.a(Singleton.cpp.o)' means the weak symbol cannot be overridden at runtime. This was likely caused by different translation units being compiled with different visibility settings.
Related to #1688.
I re-built and re-installed all dependencies with these flags and no longer see the warnings.

@kgpai thanks. This change will also help build time. However, I think if the default visibility is hidden, we will have to explicitly mark public API as default. I think we will also have to suppress this default for libraries that are not set up for default visibility hidden. That is why I assume libhdfs build is failing.
Are we able to link Velox with presto_cpp as well?
See https://stackoverflow.com/questions/3570355/c-fvisibility-hidden-fvisibility-inlines-hidden

Maybe we should start with a default visibility in this PR. Masha's PR seems to do the same. And then we can move to default hidden by marking only the public API.

Thanks Deepak, Masha.
Some points:

It seems the CI build still shows these warnings; I suspect its because the build is using the cache and not reinstalling it. I will investigate that more.
Deepak brings up a good point about how this might hide symbols; I will test this out with presto_cpp to validate that. If thats the case, then maybe we can add logic to only ensure the dependencies are built with these flags.
I tried the default visibility flags, unfortunately I did not see the warnings go away due to that. Again I will investigate more.

@majetideepak @mbasmanova
Apologies for the delay on this:

I tested these flags against presto_cpp and it compiled successfully (I thus presume they run also without problems).
It also requires that all dependencies be compiled with same flags, so on new installs its not a problem, but for e.g if you have installed folly etc previously , you will need to run setup-macos.sh again.
I tested only debug builds, so I will try a release build to further validate.


Reset current icon at end of frame

Fixes #832

SonarQube analysis reported 1 issue
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 AncientFungus:LGEPR: Cppcheck cannot find all the include files (use --check-config for details)


Handle logging of PDF docs

Adding a PDF document to an existing envelope could result in invalid
UTF-8 being passed to the logging method.  But that's ok, we just want
to handle it since we'll elide it anyway.

Rebasing / cherry-picking locally.


Library error: ElementProps: Expected React component instead of React$Component

What am I doing wrong?
class Wrapper<Child: Component<any, any>>
  extends Component<{child: Child} & ElementProps<Child>> {
  child: Child;
  childProps: ElementProps<Child>; // <-- error triggered by this line
  constructor(props) {
    super(props);
    var { child, ...rest } = this.props;
    this.child = child;
    this.childProps = rest;
  }
  render() {
    return
<child {...this.childProps}/>

  }
}

try

It works if you introduce a new type variable for the child component's props and then use that directly instead of using ElementProps:
import React, { Component } from 'react';

class Wrapper<ChildProps: Object, Child: Component<ChildProps, *>>
  extends Component<{child: Child} & ChildProps, *> {
  child: Child;
  childProps: ChildProps;
  constructor(props) {
    super(props);
    var { child, ...rest } = this.props;
    this.child = child;
    this.childProps = rest;
  }
  render() {
    return
<child {...this.childProps}/>

  }
}

Although note that the more common pattern for Higher-order components is to make a function that takes a component class and returns a new component class. This can make the code a bit clearer. There are some examples of the Flow intricacies of this in a great blog post.

Thanks, that works. I'm not using a function because jsx doesn't support <Wrapper(Chilld)/> but it does support <Wrapper child=Child/>

Great, glad it works for you! I think this issue can be closed then.