[Table] Pagination display incorrect when set number per page is 50 and total count is 158

Describe the bug
total items and total pages are wrong after changing Items per page if login Assist after clearing cache
To Reproduce
Steps to reproduce the behavior:
clear cache in Firefox or Safari or Chrome
Login Assist (IVT15), go to Admin - User tab, there are 158 items.

change Items per page from 20 to 100, got the second page. The total items is 158, which is correct.

change Items per page from 100 to 50, the total items is chagned to be 100, the total pages is 2,. The total items number should be 158 and pages number should be 3.

change items per page from 100 to 50, the total items is wrong and total pages is wrong.

Expected behavior
The total items should always be 158. the page number should be changed according to the Items per page.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):
OS: MAC
Browser: Fire, Chrome, Firefox.

seems simiar as https://github.com/carbon-design-system/carbon-addons-iot-react/issues/2156

@zhaozhuo813 Can you reproduce in Storybook? https://next.carbon-addons-iot-react.com/?path=/story/1-watson-iot-table--playground
We do not have access to the MAS Core github repo, so it's hard to know if this is coming from the application or from the component.

I can not reproduce data in Story book but I reproduce issue in storybook when I set total count = 158 and page size = 100, I got no data in table error when I turn to second page

@JordanWSmith15

I can not reproduce data in Story book but I reproduce issue in storybook when I set total count = 158 and page size = 100, I got no data in table error when I turn to second page  @JordanWSmith15

@zhaozhuo813
The reason why it shows There is no data in the story is because data object is hardcoded to only have 100 rows. There is no more data after the first page. Even we set totalItems to 158 but the actual data still has 100 rows.  I can update the story with a non hardcoded data object.
It still does not reproduce the issue you saw in your environment. totalItem refers to data.length , does data object change in your environment? Can we access your environment to debug? Thank you!

@jessieyan
Hi, I found there was something wrong in my code, so the root cause is not in carbon side, and I think this issue can be closed.
Let me close this and it would be good if you will update the story book without hardcode data. Thanks


Fix Issue #127

Fix the bug raised in issue #127

The travis build appears to be failing because of a missing Qt5 dependency; I don't know what would cause that. It's unrelated to the changes I made in this PR


Relative shortcut paths with New+

Description of the new feature / enhancement
Can New+ add the feature to create a complete path shortcut based on the relative part shortcut in New+ template. Currently the tricks to add a relative path shortcut (via another explorer thread) is not being recognize by the application when prompting the diretory. (you cannot see the relative path when "Save as"....)
Scenario when this would be used?
With relative parth shortcut, we can duplicate the shortcut per directory template
Supporting information

Hi,
Is this similar to this https://github.com/microsoft/PowerToys/issues/35792 issue? Or is this potentially another idea? And if so, would you expand on your example a bit to make sure I understand?
Thank you for sharing.

Ah, I think I finally got it! Thank you for explaining that. I didn't even realize that you could do that :-)
I think that would be possible but I haven't looked into that. I assume I will need to parse and update the .lnk file. If there is a lot of interest for this and this is approved, then I'm happy to try and work on that in the future.

Hi Christian,
Thanks for looking into this for me. There are quite some "relative path
shortcut" discussions on the internet and the work around is learnt from
there.
It would be helpful if we can parse the .lnk file to build a true MS
shortcut since the work around has some limitations.
Thank you.
On Mon, Dec 2, 2024 at 9:54 PM Christian Gaarden Gaardmark <
@.***> wrote:

Ah, I think I finally got it! Thank you for explaining that. I didn't even
realize that you could do that :-)
I think that would be possible but I haven't looked into that. I assume I
will need to parse and update the .lnk file. If there is a lot of interest
for this and this is approved, then I'm happy to try and work on that in
the future.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/35636#issuecomment-2513435978,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADTHOD3ESR4DXTBKO3ZFXJT2DUMPPAVCNFSM6AAAAABQYHNZ3KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJTGQZTKOJXHA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Add public set function

This PR aims to add a public set function to Fuse so the internal list Fuse searches against can be changed without having to instantiate a whole new Fuse object each time. I thought this might also go well with krisk#46 but provide an extra option where you may not care about manipulating the actual list, and just want to declaratively set it to a given array of things.
Feel free to add comments or let me know what I can change to make it better.

Tests pass locally, Travis error seems to be from something related to the npm install


remove custom Promise handling

All Node versions we currently support (Node 4, 6 and 8) have built-in Promise support, so we plan to remove the Promise options from the Constructor.
This like this will have no effect after v13.0.0
const Client = require('./../lib/index')
const github = new Client({
  Promise: Promise
})

Please let me know if this causes any trouble for you, maybe we can find a workaround that we can add to the migration guide :)
Related discussions are at #262, #258, #232

This is done in https://github.com/octokit/node-github/pull/629 and will be closed once it's merged into master


Sync up PR #43 to v1.0-dev after it was merged into master

Performed a git merge upstream/master while on upstream/v1.0-dev branch.
Had to resolve some conflicts because files where moved around.

@markdirish
I added a commit to fix lint issues.


Need to be able to output to Windows metafiles and the like

Reported by hankin on 3 Sep 2002 15:47 UTC
The PLOT+ "metaview" program might be modified for this purpose
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/425

Comment by<EMAIL_ADDRESS>on 21 Oct 2003 20:15 UTC
Moving from the<EMAIL_ADDRESS>user to the<EMAIL_ADDRESS>user.

Comment by<EMAIL_ADDRESS>on 5 Nov 2003 22:56 UTC
move to Ferret Tasks, assign to Ansley

Comment by<EMAIL_ADDRESS>on 10 Nov 2003 21:43 UTC
talk with GFDL

Comment by<EMAIL_ADDRESS>on 5 Nov 2004 19:20 UTC
Changing Component of bugs in "Ferret Tasks" from All to Main


Hotfix

Defects found in previous release.

Summary by CodeRabbit

New Features

Enhanced debugging capabilities with a logging mechanism for tracking script execution.
Introduced a temporary file for merged content before clipboard operations.
Added alternative methods for clipboard copying to ensure successful operations.
User feedback is provided through command line messages upon completion, including total files processed and character counts.

Bug Fixes

Improved error handling during clipboard operations.

Chores

Implemented cleanup of temporary files after execution.

@coderabbitai review

@coderabbitai approved?


Add a fixer that replaces uses of homoglyphs in code names

This is useful for programmers who often have multiple active
keyboard layouts and might accidentally name things with homoglyphs
in without realising
Homoglyphs are unicode characters that look very similar to basic ascii
characters.
You might start typing using cyrillic keyboard layout, backspace when you
see your mistake, but leave a С character, which looks just like a C, but is actually
a cyrillic letter for 'S sound'.
The problem is compounded by IDE autocomplete, with works just fine with this
badly named class/variable/etc. And even PSR-0/4 autoloading, which works fine,
until you hit a filesystem that doens't work well with unicode characters and everything
breaks.

I think that idea itself is nice ;)
what about space homoglyphs ? there are few spaces in UTF ;d

I'm was focusing on homoglyphs that you might have in code, so class names or variable names. I think it would be inappropriate to start editing literal strings in code.
The point was to change symbols you might not notice. Given that spaces are not normally part of a name these would be noticeable.

Just having a look, you can put weird spaces in a name, this is valid, but its pretty noticeable.
$replac ements = ...;

Weird spaces in general, instead of normal spaces, and php cannot parse the file, so I would say its not an issue.

Thank you @mcfedr.

@keradus Thank you for merging me


Integrated doc HTML and spec JSON into one request

Is there a way to split this back into 2 request? We use a common UI for many swagger docs and that requires a discrete json file.


Remove SHARD env that was overriding matrix

In #20077 I switched to using matrix in the Windows/macOS Cirrus configs to reduce duplicate. I left a (seemingly redundant) SHARD env variable at the top of the config, but it looks like that one was used, so the Windows tools tests have been running the non-tools tests since 🙈
This removes that extra SHARD env variable so it should come from the matrix (this also feels like a Cirrus bug, so once the build has run and I've confirmed fixed, I'll mentioned it to them).

Before (see the chips on top-left and bottom-right):

After:


[Bug] catmanga returns 502 gateway errors

Plugin
catmanga
Describe the bug
searching for nonexistent manga results in a 502 error and unexpected logout
To Reproduce
Steps to reproduce the behavior:

open mango plugins, select catmanga
search for any manga title from catmanga (cafe for instance)
results are returned
search for a manga title not on catmanga (nagatoro for instance)
502 popup is returned from mango
navigate away from the search page
see login screen

Expected behavior
graceful search failure
Screenshots
If applicable, add screenshots to help explain your problem.
Mango version
v0.21 in arm64v8 docker container
Additional context
similar search with another engine, manganelo, worked ok, so i dont believe it to be the install's fault. This is returned in the chrome developer console when the failure is experienced:
http://mango.xxxxx.xx/api/admin/plugin/list?query=nanafuku&plugin=Catmanga 502 (Bad Gateway)

This is likely not an issue with Mango or the plugin. CatManga uses Cloudflare, and the 502 error you see is due to the fact that Cloudflare couldn't establish a connection to the upstream server. See https://support.cloudflare.com/hc/en-us/articles/115003011431-Troubleshooting-Cloudflare-5XX-errors#502504error

I see; the catmanga plugin does require some careful use, which could relate to a bug in mango proper, but I'll let you make that call.
If a title is searched for that exists in catmanga, such as "cafe" (the site's title for goddess cafe terrace), then results are returned correctly, and everything is fine. Even fuzzy matching works, if I search for "goddess".
If I search for another word, such as "nagatoro" or some other manga not found on catmanga, then a 502 is encountered. This is sort of also to be expected, since the instructions for catmanga say to search for a title from the site. However, this could be more gracefully handled. I say this because the actual issue I face from this 502, which I have modified the report to include, is that I am then logged out of mango unexpectedly. Nothing else can be done with the session once the 502 is returned. I can do another search in the same window, but the error message is still risen (though it says 200 OK if a successful search is made). Navigating away from the search page causes a new login prompt.

If I search for another word, such as "nagatoro" or some other manga not found on catmanga, then a 502 is encountered.

I can't reproduce this. See the screenshot below

I say this because the actual issue I face from this 502, which I have modified the report to include, is that I am then logged out of mango unexpectedly. Nothing else can be done with the session once the 502 is returned. I can do another search in the same window, but the error message is still risen (though it says 200 OK if a successful search is made). Navigating away from the search page causes a new login prompt.

Hmm, this is strange. Do you see any error messages in your console (from the docker container, not your browser console)?

Since you mentioned you are using Nginx in https://github.com/hkalexling/Mango/issues/190, I guess the 502 you are seeing is from Nginx and not Cloudflare.

Correct, the 502 is coming from my upstream nginx proxy. The error from the proxy directly in front of the mango install is:
2021/05/01 03:55:23 [error] 15522#15522: *3518 upstream prematurely closed connection while reading response header from upstream, client: <IP_ADDRESS>, server: mango.xxxxxx.xx, request: "GET /api/admin/plugin/list?query=Naga%20Toro&plugin=Catmanga HTTP/1.0", upstream: "http://<IP_ADDRESS>:9000/api/admin/plugin/list?query=Naga%20Toro&plugin=Catmanga", host: "mango.xxxxxx.xxx", referrer: "http:// mango.xxxxxx.xxx/download/plugins?plugin=Catmanga"

I did test this as well with the direct ip:port, and got the following:
Search failed. Error: [0] error
...and the same logout behavior as through nginx. There was no additional output when checking the docker container console.

I was not able to reproduce this with a direct ip:port to the docker container. I tried adding an nginx container and I am still unable to.
I followed the steps you mentioned,

1. open mango plugins, select catmanga
2. search for any manga title from catmanga (cafe for instance)
3. results are returned
4. search for a manga title not on catmanga (nagatoro for instance)
5. Error popup is returned from mango
6. navigate away from the search page
7. see login screen

Except I never saw the 502 error or got sent back to the login screen.

After getting the error I can navigate back the the library just fine, and I get the following lines in the nginx logs.
reverse    | <IP_ADDRESS> - - [20/May/2021:09:30:42 +0000] "GET /api/admin/plugin/list?query=cafe&plugin=Catmanga HTTP/1.1" 200 907 "http://mango.dev.lan/download/plugins?plugin=Catmanga" "Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:88.0) Gecko/20100101 Firefox/88.0" "-"
reverse    | <IP_ADDRESS> - - [20/May/2021:09:31:07 +0000] "GET /api/admin/plugin/list?query=nagatoro&plugin=Catmanga HTTP/1.1" 200 69 "http://mango.dev.lan/download/plugins?plugin=Catmanga" "Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:88.0) Gecko/20100101 Firefox/88.0" "-"
reverse    | <IP_ADDRESS> - - [20/May/2021:09:32:11 +0000] "GET /library HTTP/1.1" 200 57630 "http://mango.dev.lan/download/plugins?plugin=Catmanga" "Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:88.0) Gecko/20100101 Firefox/88.0" "-"
...

I load up directly via port/ip again, and load in the api request into my browser:
http://<IP_ADDRESS>:9000/api/admin/plugin/list?query=komi&plugin=Manganelo
...and I am next met with a login page after a few seconds. If I enter my credentials, I am directed to:
http://<IP_ADDRESS>:9000/api/admin/plugin/list
I am also able to load up results correctly with Catmanga plugin, but only if a true keyword is used like 'cafe' or 'komi', rather than a search term.
Request headers from Catmanga calls are generally identical for successful or non-successful searches, except that the successful ones are 200, and the unsuccessful ones are 302 redirects with a target of /login, and have 0 response length. Is there another location where output from these calls is stored? I see a log level in the mango config file, but am not sure where this is sent.

i altered the log level to debug and got this info when attempting a search:
May 28 14:53:42 rpi4 docker[20334]: [DEBUG]   2021/05/28 14:53:42 | 302 GET /api/admin/plugin/list?query=cafe&plugin=Catmanga 208.04µs
May 28 14:53:42 rpi4 docker[20334]: [DEBUG]   2021/05/28 14:53:42 | 200 GET /login 77.28µs
i instead tried the direct ip:port to watch what happened, and got the 'error 0 error' issue. what i found was that the container restarted itself after a segfault. here is a successful search first, and then an unsuccessful search right after:
May 28 14:55:17 rpi4 docker[20334]: [DEBUG]   2021/05/28 14:55:17 | Setting proxy
May 28 14:55:17 rpi4 docker[20334]: [DEBUG]   2021/05/28 14:55:17 | Setting read timeout
May 28 14:55:17 rpi4 docker[20334]: [DEBUG]   2021/05/28 14:55:17 | Requesting https://catmanga.org/
May 28 14:55:17 rpi4 docker[20334]: [DEBUG]   2021/05/28 14:55:17 | 200 GET /api/admin/plugin/list?query=cafe&plugin=Catmanga 218.68ms
May 28 14:55:28 rpi4 docker[20334]: [DEBUG]   2021/05/28 14:55:28 | Setting proxy
May 28 14:55:28 rpi4 docker[20334]: [DEBUG]   2021/05/28 14:55:28 | Setting read timeout
May 28 14:55:28 rpi4 docker[20334]: [DEBUG]   2021/05/28 14:55:28 | Requesting https://catmanga.org/
May 28 14:55:28 rpi4 docker[20334]: Failed to raise an exception: 501243200
May 28 14:55:28 rpi4 docker[20334]: [0xaaaae1dd5ad0] ???
May 28 14:55:28 rpi4 docker[20334]: [0xaaaae1bee24c] __crystal_raise +36
May 28 14:55:28 rpi4 docker[20334]: [0xaaaae1cf8628] ???
May 28 14:55:28 rpi4 docker[20334]: [0xaaaae1cf857c] ???
May 28 14:55:28 rpi4 docker[20334]: [0xaaaae1eac5b8] ???
May 28 14:55:28 rpi4 dockerd[897]: time="2021-05-28T14:55:28.340083119Z" level=info msg="ignoring event" container=751dc42bac6ff38bd9372b192e1b2a55a3d6a8f08ddd1bcd0b8d8ab243a74d44 module=libcontainerd namespace=moby topic=/tasks/delete type="*events.TaskDelete"
May 28 14:55:28 rpi4 containerd[727]: time="2021-05-28T14:55:28.340251043Z" level=info msg="shim disconnected" id=751dc42bac6ff38bd9372b192e1b2a55a3d6a8f08ddd1bcd0b8d8ab243a74d44
May 28 14:55:28 rpi4 containerd[727]: time="2021-05-28T14:55:28.340446040Z" level=error msg="copy shim log" error="read /proc/self/fd/10: file already closed"
many more lines follow regarding the restart of the container, which i can pastebin if need be.

@ch604 Thanks for taking the time to investigate! Yes it looks like the process crashed when sending request to catmanga. Since we can't reproduce the bug on AMD64, I guess it's related ARM. Let me try it on my RPi and see if I can reproduce there.

I've confirmed the issue on ARM32. Apparently Mango would crash on ARM whenever a plugin calls mango.raise, a helper function provided by Mango. In your case, because no matching manga is found, the plugin calls the function and crashes Mango.
https://github.com/hkalexling/mango-plugins/blob/8119decdec3c829427fc8b92d2f414020691ad98/plugins/catmanga/index.js#L55-L57
This might be a bug in duktape.cr and I will submit an issue there. Since this issue is not related to the catmanga plugin, I am transferring it to the main repo for Mango.


LICENSE file missing

Not available LICENSE file in source directory structure
Please. Added license and copyright notice.
the fedora pakaging guideline is very strictly precise about this problem
https://fedoraproject.org/wiki/Packaging:LicensingGuidelines?rd=Packaging/LicensingGuidelines#License_Text
thanks
regards

Done via https://github.com/mttkay/signpost/commit/0134fe9192f7a33f888632535367dc22cc305a88


How to extends default CSS class to the custom Widget?

Hi,
I am creating my own custom widget using the "radiogroup" class but I am facing an issue regarding the default CSS because my new widget does not have those default CSS classes.
Survey.JsonObject.metaData.addClass("customradio", [], null, "radiogroup")
Now, there is a new "Customradio" widget created but it's not getting the default radiogroup classes. I have searched and found that we can use " model.css = css;" and apply those classes at the run time. But I want a solution that how can we set those classes while creating a widget? In widget creating code.

Here is the custom widget live sample - https://plnkr.co/edit/HdnYE5?p=preview
You can use the following option:
  //If you want to use the default question rendering then set this property to true. We do not need any default rendering, we will use our our htmlTemplate
  isDefaultRender: false,

I do not want to create my own HTML structure. I just want all the default CSS class to be available through the widget. For example to my above code.
var css = {  "customradio": { "root": "sv_qcbc form-inline",                         "item": "radio",                         "itemChecked": "checked",                         "itemInline": "sv_q_radiogroup",                         "label": "",                         "itemControl": "",                         "itemDecorator": "sv-hidden",                         "controlLabel": "",                         "materialDecorator": "circle",                         "other": "sv_q_radiogroup_other form-control",                         "clearButton": "sv_q_radiogroup_clear button",                         "column": "sv_q_select_column"}}  var model = new Survey.Model({surveyId: surveyId, surveyPostId: surveyId});                 model.css = css;
I want to achieve the above result through widget only.

Sorry, but afraid I don't understand your scenario. Please share a minimal working sample illustrating your question.


Broken search

Describe the bug
Search no longer works after the last update
To Reproduce
Steps to reproduce the behavior:

Go to Docs
Click on Serach bar
Enter query
See 'Loading' forever

Expected behavior
Return search result
Screenshots

Desktop (please complete the following information):

OS: Debian Linux
Browser chrome
Platform: Chrome extension,
Version 4.0.10

Additional context
Add any other context about the problem here.

This is happening to me too.

OS: macOS Big Sur (11.6)
Version 4.0.10


Running into error: Unknown plugin scope https

Running moon build returns this:

proto install
pnpm install
..

> moon-examples@ moon /Users/marvin/Developer/_TMP/examples
> moon --color --log debug "run" "angular-app:build"

[DEBUG 2024-12-05 14:30:51.379] moon_cli  Running moon v1.25.0 (with ~/Developer/_TMP/examples/node_modules/.pnpm/@moonrepo+cli@1.25.0/node_modules/@moonrepo/cli/moon)  args=["/Users/marvin/Developer/_TMP/examples/node_modules/.pnpm/@moonrepo+cli@1.25.0/node_modules/@moonrepo/cli/moon", "--color", "--log", "debug", "run", "angular-app:build"]
[DEBUG 14:30:51.382] moon_env:load_environments  Creating moon environment, detecting store  store="/Users/marvin/.moon"
[DEBUG 14:30:51.382] proto_core::proto:load_environments  Creating proto environment, detecting store  store="/Users/marvin/.proto"
[DEBUG 14:30:51.383] moon_workspace::workspace:load_workspace  Attempting to find workspace root  working_dir="/Users/marvin/Developer/_TMP/examples"
[DEBUG 14:30:51.383] moon_workspace::workspace:load_workspace  Found a potential workspace root  workspace_root="/Users/marvin/Developer/_TMP/examples"
[DEBUG 14:30:51.383] moon_workspace::workspace:load_workspace  Creating workspace  workspace_root="/Users/marvin/Developer/_TMP/examples" working_dir="/Users/marvin/Developer/_TMP/examples"
[DEBUG 14:30:51.383] moon_workspace::workspace:load_workspace  Loading .moon/workspace.yml  workspace_root="/Users/marvin/Developer/_TMP/examples"
[DEBUG 14:30:51.386] proto_core::proto_config:load_workspace:load_config  Loading .prototools  file="/Users/marvin/.proto/.prototools"
Error: config::parse::failed

  × Failed to parse ProtoConfig.
  ╰─▶   × plugins.moon: Unknown plugin scope `https`.
         ╭─[8:8]
       7 │ [plugins]
       8 │ moon = "https://raw.githubusercontent.com/moonrepo/moon/master/proto-plugin.toml"
         ·        ─────────────────────────────────────┬────────────────────────────────────
         ·                                             ╰── Fix this
         ╰────

 ELIFECYCLE  Command failed with exit code 1.
~/Developer/_TMP/examples on  master [!] at 14:30:51
bat /Users/marvin/.proto/.prototools
───────┬────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
       │ File: /Users/marvin/.proto/.prototools
───────┼────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
   1   │ moon = "1.29.4"
   2   │ node = "20.13.1"
   3   │ npm = "10.9.0"
   4   │ pnpm = "9.0.2"
   5   │ yarn = "4.5.1"
   6   │
   7   │ [plugins]
   8   │ moon = "https://raw.githubusercontent.com/moonrepo/moon/master/proto-plugin.toml"

It works in my projects tho but not in the example repo

Oh wait that comes from my home config, but why is that a problem anyway?

@muuvmuuv This repo is using an old moon version, which is using an old proto version. The proto version differs from the one in your home directory, so it's resulting in this error.

Ohh yeah, makes sense, I solved it by only define plugins in projects themself, which also makes more sense so other authors have them as well. Closing :) btw I really love moon and proto works very well together and the experience is awesome, especially together with CI just missing a feature to call multiple tasks in parallel from CLI like with npm-run-all.


It seems that switch network is not available

It seems that switch network is not available

@morandalex Hmm, may I ask which version of usedapp do you have installed?

  @usedapp/core": "^0.4.1


SAS Viya naming updates

There are some SAS Viya offering name changes that require updates to names used in this project.

The only changes are minor updates to main README made by @FredPerrySAS.

This issue addressed by Release 1.3.1


Any ARM version for Pi?

I'm trying to run it on a Rpi but get:

pi@my-pi:~ $ sudo docker run -e BASIC_AUTH_USER=pi -e 'BASIC_AUTH_PWD=gfhgfhgf' -d -p 8000:8000 alseambusher/crontab-ui:latest
WARNING: The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm/v7) and no specific platform was requested

Seems to be fixed now, I am running it on my Pi without an issue.


Create validation of env variables in comparison to rules declared in JSON file

Goal is to create validation of set ENV variables and rules described in config JSON.
For example if ENV is set to true but required params are not present inside JSON app should fail with displaying (logging) error.
Example:
export URI_VALIDATION_ENABLED=true

but inside rules_config.json
...
  "uri_validation_enabled": false,
...

Should be noticed in log file and then application should be closed with exit status other than 0

After latest commit 2013b09 only thing left is to write logic for check_env_config_conflicts() inside core/config_compliance_check.py.

Get every *_ENABLED env variable
Get every *_enabled variable from rules_config.json

Compare in smart way and call sys.exit(1) with print of approperiate message if conflicts detected.


Build on amd64 and arm64

Building binaries for arm64 might help on new Apple Silicon (m1) environments.

Pull Request Test Coverage Report for Build 57

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 55:
0.0%

Covered Lines:
1

Relevant Lines:
1

💛  - Coveralls


M3-3783: Replace react-async-script-loader package w/ react-async-script

Description
Replace react-async-script-loader, which hasn't been updated in several years, with react-async-script.
The only place this was being used was the Payment drawer. I don't believe that react-async-script passes props as react-async-script-loader did, so this will require some adjusting to keep working as intended.
Type of Change

Non breaking change ('update', 'change')

To-Do

[x] Further tweaking and testing with react-async-script to ensure this is actually working

https://github.com/linode/manager/blob/97801834f1eb5713445bba7b1de4ea22d5406903/packages/manager/src/features/Billing/BillingPanels/BillingSummary/PaymentDrawer/PaymentDrawer.tsx#L74
When you test, if you change the paypalScriptSrc() bit to some other random URL, the button should stop showing up (because isPaypalScriptLoaded should not get set to true in that case). So I believe this should be working as intended now, please let me know if it is not

@Jskobos I was hoping to find a way to access the async wrapper's error state to make things simple, but the recent changes are in lieu of that... alternatively I can just hide the whole PayPal section if isPayPalInitialized is false.


Eval gpt2 fails with CUBLAS_STATUS_NOT_INITIALIZED

Running
python eval/eval.py eval/yamls/hf_eval.yaml icl_tasks=eval/yamls/winograd.yaml model_name_or_path=gpt2

fails with
RuntimeError: CUDA error: CUBLAS_STATUS_NOT_INITIALIZED when calling `cublasCreate(handle)`

The same command runs fine with other models (e.g., EleutherAI/gpt-neo-125M).
Any ideas what could be going wrong in the case of gpt2?
Environment

System Environment Report
Created: 2023-06-22 15:38:44 UTC
PyTorch information
PyTorch version: 1.13.1+cu117
Is debug build: False
CUDA used to build PyTorch: 11.7
ROCM used to build PyTorch: N/A
OS: Ubuntu 20.04.6 LTS (x86_64)
GCC version: (Ubuntu 9.4.0-1ubuntu1~20.04.1) 9.4.0
Clang version: Could not collect
CMake version: version 3.26.3
Libc version: glibc-2.31
Python version: 3.10.11 (main, Apr  5 2023, 14:15:10) [GCC 9.4.0] (64-bit runtime)
Python platform: Linux-5.19.0-1026-gcp-x86_64-with-glibc2.31
Is CUDA available: True
CUDA runtime version: 11.7.99
CUDA_MODULE_LOADING set to: LAZY
GPU models and configuration:
GPU 0: NVIDIA A100-SXM4-40GB
GPU 1: NVIDIA A100-SXM4-40GB
Nvidia driver version: 520.61.05
cuDNN version: Probably one of the following:
/usr/lib/x86_64-linux-gnu/libcudnn.so.8.5.0
/usr/lib/x86_64-linux-gnu/libcudnn_adv_infer.so.8.5.0
/usr/lib/x86_64-linux-gnu/libcudnn_adv_train.so.8.5.0
/usr/lib/x86_64-linux-gnu/libcudnn_cnn_infer.so.8.5.0
/usr/lib/x86_64-linux-gnu/libcudnn_cnn_train.so.8.5.0
/usr/lib/x86_64-linux-gnu/libcudnn_ops_infer.so.8.5.0
/usr/lib/x86_64-linux-gnu/libcudnn_ops_train.so.8.5.0
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True
Versions of relevant libraries:
[pip3] numpy==1.24.2
[pip3] pytorch-ranger==0.1.1
[pip3] torch==1.13.1+cu117
[pip3] torch-optimizer==0.3.0
[pip3] torchmetrics==0.11.3
[pip3] torchtext==0.14.1
[pip3] torchvision==0.14.1+cu117
[conda] Could not collect
Composer information
Composer version: 0.14.1
Composer commit hash: None
Host processor model name: Intel(R) Xeon(R) CPU @ 2.20GHz
Host processor core count: 12
Number of nodes: 1
Accelerator model name: NVIDIA A100-SXM4-40GB
Accelerators per node: 1
CUDA Device Count: 2

Can you check whether this happens with the composer launcher?
(replace python with composer in the command you pasted above)

Yes, same error with composer.
(Running through Docker image mosaicml/llm-foundry:1.13.1_cu117-latest.)

Are you still encountering this issue on main?

Hey, I'm currently not able to test this. Feel free to close this issue if you can't reproduce this.

Closing, please feel free to open a new issue if you are still encountering problems.


Error and condition reporting strategy

The MIM core currently follows a somewhat unusual pattern for reporting errors, warning conditions, and success.  To wit, for each kind of message, if that event occurs, it appends a descriptive message to a file on disk with a suggestive name.
The conventional thing for a program of this level of complexity is to write error and warning messages to the standard error stream, and routine notifications (such as progress and success reports) to the standard output stream.  By default those are interleaved in the controlling terminal, but may be redirected separately.

Pro: This arrangement is very strongly conventional for command line tools, especially batch-mode ones with low interface complexity.  Conventionality is itself an advantage.
Pro: Different types of messages will be interleaved in order of occurrence, preserving that information.  This may help the user diagnose any problems.
Pro: The user interface is simpler: no need to describe what file names to look for.
Con: Diagnostic output goes to an ephemeral location (the terminal) by default; but there are standard tools for controlling this (namely, shell redirection, tee, or nohup).  We can mention these in the user manual, if appropriate.
Con: It is not apparent at a glance which conditions occurred during a run, as those lines may be buried in subsequent output.  This can be mitigated by printing a run summary at the end, or by the user applying grep to a saved log file.

@edoddridge: I am compelled to ask, why did you code it the way it currently is?  Did I miss a consideration?

It's coded this way because I added the warnings and error messages in a piecemeal fashion as I was using the code and debugging simulations. The lack of coherent strategy and the flaunting of convention are not deliberate choices that were carefully considered.
Some models produce stdout and stderr  files by default to which the various outputs get redirected. The runtime switches then determine the level of output sent to these files - from virtually nothing to many diagnostics that substantially slow down the simulation.

Yes.  Controlling logging by switch is the next level of sophistication, which can be added incrementally.

Specific notifications the model currently emits:

[ ] run completion at final time step.  Currently goes to file, should go to stdout.
[ ] successful end of run.  Currently goes to stdout, as it should; but may be a duplicate.
[ ] layer too thin warning.  Currently goes to file, should go to stderr.
[ ] NaN detected error.  Currently goes to file and stdout, should go to stderr.
[ ] depths must be positive error.  Currently goes to stdout, should go to stderr.
[ ] maximum SOR iterations exceeded.  Currently goes to stdout, should go to stderr.
[ ] unbalanced free surface warning.  Currently goes to stdout, should go to stderr.


ModuleNotFoundError: No module named 'triton'

May I ask if anyone has encountered such a problem? The 'triton' module is missing, but the installation will report an error.

works fine with triton error

works fine with triton error

yeah， I just realized that

how to fix it

On windows, you need to install triton manually

Download the trition whl from https://huggingface.co/madbuda/triton-windows-builds, and then pip install this manually

Download the trition whl from https://huggingface.co/madbuda/triton-windows-builds, and then pip install this manually

thanks，I'll try

Anyone know whether triton is needed in some cases ? Should I ignore this package?


Add coloring table cells by value

It would be awesome to be able to color cells in a table by value. For example, specifying one column that 0% is dark red, 0-50% is bright red, 50-80% is orange, 80-90% is yellow and 90%+ is green. Another may be more arbitrary, 0-5 red, 5-10 yellow, 10-15 green.
I have been kludging together html reports for coming up on 10 years using ggplot2 and xtable and coming up with clever regular expressions to color cells and I'm excited that this may be a far more easily maintained, far more modular and far quicker to develop methodology.

I can do this in HTML.  In RTF, I can control bolding by cell.  I've got all this cell styling in the queue.  I've wanted to do it for a long time.  Let me send you an example with HTML.  Not sure when the other report formats will be accomplished.  Every format is different, so they all have to be done independently. PDF is particularly difficult.  Thanks for your comment.  It helps me to know what people are looking for.

Somehow I thought I could do this in HTML.  But now that I investigate, I think it was just something I was planning but never implemented.  Sorry about that.  I will up the priority on this feature.  The best I can do currently is conditional bolding:
library(reporter)

# Create temporary path
tmp <- file.path(tempdir(), "example12k.rtf")

# Prepare data
df <- data.frame(names = rownames(mtcars), mtcars[, 1:3])

# Set style indicator variable
df$mpgind <- ifelse(df$mpg > 20, TRUE, FALSE)

# Create table
tbl <- create_table(df, first_row_blank = TRUE,
                    header_bold = TRUE, borders = c("top", "bottom")) %>%
  define(names, label = "Car Name") %>%
  define(mpg, label = "Miles Per Gallon",
         style = cell_style(bold = TRUE, indicator = "mpgind")) %>%
  define(cyl, label = "Cylinders") %>%
  define(disp, label = "Displacement")  %>%
  define(mpgind, visible = FALSE) %>%
  titles("Table 1.0", "MTCARS with Indicator Variable",
         borders = "none", bold = TRUE, font_size = 11) %>%
  footnotes("* Motor Trend, 1974", borders = "none", blank_row = "none")

# Create report and add custom style
rpt <- create_report(tmp, output_type = "RTF", font = "Arial") %>%
  add_content(tbl)

# Write out report
write_report(rpt)

# View report
# file.show(tmp)

Thank you for thinking about it and considering it in the future.
For the record, the package flextable does have formatting for data frames. I don't know if that would end up being a straightforward data type to implement or a path you'd be interested in going in.

Moved this to reporter package #326


How to subscribe event from Javascript code to Host Code

Use in case when user input data in Host Code application , I want to send event to Javascript to check validation data.
I had search on google but same question not found.
Thank for help.

Thank you very much. working, amazing........


Eval geomodels with tests

The output results do not look too good -
Metrics:
Exact accuracy : [0.00]
mean error [1999.89],
median error [1510.49]
max error [6962.15]
AUC of error curve [0.43]
Am I reading the .pt file incorrectly? The 'true_points_list' and 'pred_points_list' should have a 1-1 mapping right?

max on the prediction to get the actual prdicted label -- ok. Lets do it in the load_metrics function as discussed offline. Will merge.


[BUG] non-ascii characters are broken in Simple JSON protocol

non-ascii characters in string are broken when encoding struct to json using SimpleJsonProtocol.
   "autocomplete_suggestion_display_strings": [
      "San Francisco, California",
      "Beijing � z\/�1P",
      "Montreal � 1|�Plus?�",
      "New York � S�",
      "\u0005� � S�",
      "�� � S�",

After search, I find that the cause is casting java character directly to byte in this method:
    @Override
    public void writeString(String str) throws IOException {
        writeContext().beforeWrite();

        int len = str.length();
        UnsafeByteArrayOutputStream baos = new UnsafeByteArrayOutputStream(len);

...
            } else {
                baos.write(c);
                           ^ this method accepts a char but directly cast to byte and store into a bytearray
            }
        }

...
    }

public synchronized void write(int b) {
        ensureCapacity(count + 1);
        buf[count] = (byte) b;
                       ^ direct cast happens here
        count += 1;
    }

I suggest to use a utf8 encoder to encode char to bytes

Thanks for the report!  I'm sorry to say that I can't tell what the original characters should be.  Would you please let me know what kind of input results in the broken output?
I can definitely see in the code that we aren't handling non-ASCII characters in this class.  UTF-8 encoding seems like the wrong approach to me, as json strings are strings of characters, not strings of bytes.  We ought to be able to just use \uXXXX escape sequences.

Well, thinking about it more, it does seem like we can lean on UTF-8 here.  Transports deal in bytes, not characters, so the Protocol is in charge of coding in this case.  I should have a fix soon.

I've just merged a fix; it should be available now in snapshot builds, and will be in the final 2.0.0 release whenever I get around to publishing it.

Thanks for fixing this!


feat: Auto detect dark theme

What

Let's user change the theme according to the device theme
Also need help in the file lib/pages/user_preferences_settings.dart as a comment left on line 84

https://user-images.githubusercontent.com/57723319/159159229-43fe95f2-ac6d-406b-860c-c03bddf1a24c.mp4
Fixes bug(s)

#1257

@AshAman999 adding 'feat: ' in the title makes your PR semantic, and thus Release Please will generate the Changelog automatically. I just edited the title accordingly.

Can it be possible that the test cases were written into taking consideration according to the previous code, so even if my code is doing the work it is failing the tests because it wasn't supposed to be for it? @M123-dev

@M123-dev  did the changes, you can have a look at the recent changes

@M123-dev id the changes, please check again,
sorry for the troubles, feels like there is a lot of ongoings for a simple dark theme implementation ;(

No need to apologize, now after the small touch ups the code looks great.Only the screenshot tests for the dark preferences pages (left tab) have over 80% deviation.  It seems there is some problem

No need to apologize, now after the small touch ups the code looks great.Only the screenshot tests for the dark preferences pages (left tab) have over 80% deviation. It seems there is some problem

Though i may be wrong , is it possible that the tests were written so as to be for the previous code, since we are using a different method to chose dark theme so it may be possible that the code might be doing okay but the tests needs to be rewritten

That's a likely answer, I just ran your PR and it worked fine for me. Merging is blocken as long as the tests are failing. I'll have a look at the screenshot test later

Heyy @AshAman999 I just checked the implementation here: https://github.com/openfoodfacts/smooth-app/blob/develop/packages/smooth_app/test/pages/user_preferences_page_test.dart#L32
There we create mock SharedPreferences with the key themeDark to either true or false.
This should be adopted for your New currentTheme String with the options Dark and Light. You can leave out System Default as it's always Light by default.

Heyy @AshAman999 fixed the screenshot test + resolved conflicts

I am looking at the changes you did, needed to see how exactly how it was done.
Thanks for the rescue here, had nearly given up here

Yeah the screenshot tests can be sometimes confusing and yes I think so, didn't do any major changes outside of the tests and we've been waiting for the automatic darkmode for way too long now

But yeah thanks a lot for this great new feature

Really happy to see this pr merged :)
thanks a lot for the guidance @M123-dev :100:


Font-color not changed in graph contextual popups when in dark mode #621

Hotfix over the light/dark mode on pop ups mentioned on issue #621
Before:

After:

@alfredorubin96 Two things:

Default text color should never be a constant color (it should also be a Needle Token)
It should be checked if this behavior can be done once for all the portals (every modal has this problem)


cln: Unable to connect, no address known for peer

It is not the first time that I see this issue around lnprototest, but each time I was not able to reproduce it, I will put it there and see if we could have some useful log in the future
https://github.com/rustyrussell/lnprototest/pull/65#issuecomment-1298619664

this should be related to https://github.com/ElementsProject/lightning/issues/5683


[RFC] Add Evaluation chapter

Branched from #48.

I believe this chapter is now done. Two questions:

do I have to explain what SIG is?
Do I have to discuss anything else in the development methodologies? E.g. how has Java suited us, or the testing tools, or maven, or how did the PRs go, or ...
Eventually this chapter could refer back to earlier chapters, but at the time of writing those do not yet exist.

I still have to process https://github.com/spoofax-shell/doc/pull/47#discussion-diff-67040410 and perhaps update this when I have rewritten the problem analysis section.

No.
No.

I do think you have to elaborate more on our sources of delay.
You could explain more explicitly that the whole interpreter has been changed during our project, and therefore we had to shift part of our development effort to interoperating with  the new architecture for ~2 weeks.
While explaining that, you could also give it a positive twist and include in the evaluation that a significant amount of code was contributed to upstream, thereby illustrating that Spoofax in fact still benefitted from our delay.

Good feedback, I will update that section.

You could explain more explicitly that the whole interpreter has been changed during our project, and therefore we had to shift part of our development effort to interoperating with the new architecture for ~2 weeks.
While explaining that, you could also give it a positive twist and include in the evaluation that a significant amount of code was contributed to upstream, thereby illustrating that Spoofax in fact still benefitted from our delay.

This 1000x

I rebased this branch onto master so that it's not dependent on #47 and #48 anymore.


Log scale for scatter plot

Offer users to check a box to log scale their scatter plot.
Plot would be refreshed upon clicking

done


Module reaarange for GWT support

I found, that gdx-pay is not ready to be used with GWT.
A project using gdx-pay just can't be compiled with GWT, because of using java reflections in PurchaseSystem.
Libgdx provides ClassReflection utility to use reflections in cross-platform way. But in this case gdx-pay module will be dependent from libgdx, and then gdx-pay-server will also depend from libgdx.
So I suggest to add additional module: gdx-pay-common, which will contain common classes for gdx-pay and gdx-pay-server, but will not depend from libgdx. So, gdx-pay-server will not depend from gdx-pay, but only from gdx-common. And gdx-pay will depend from libgdx. That will also simplify PurchaseSystem code. However it will make it impossible to use gdx-pay without libgdx.
What do you think? Once you agree on the changes, I will implement them.

 And then add a gdx-pay-common which is used both for server & client!?
I believe that's a good idea, but I would change the names:

gdx-pay-client: the client-only functionality (what you call "gdx-pay" in your suggestion). It extends from libgdx and can contain reflection code. Rather than calling it "gdx-pay", I would prefer to adding a "client" to make it clear it's for the client (i.e. libgdx-app).
gdx-pay: Ditto now except reflection code. Rather than calling it "gdx-pay-common" as you proposed, I would keep it as is. We can call it "gdx-pay-common" as well if you prefer?

In any case, all we would add is a gdx-pay-client which extends libgdx and is used by the clients. We can move relevant stuff over from gdx-pay to gdx-pay-client. If everyone prefers that, we can finally rename gdx-pay to gdx-pay-common.
I believe that would work nicely? Comments?

That was my first idea also. I was just in doubt because renaming the module could cause troubles for existing users. But if there will be a message on the main page, then I would prefer to go with what you have described: gdx-pay, gdx-pay-client. I agree, this is more clear and more in style of libgdx.
In this case I guess the change should be released in 0.7.0-SNAPSHOT, to do not cause troubles for 0.6.0-SNAPSHOT users. Should I change the version in my pull request, or will you handle this somehow?

Great! I'd say let's just keep 0.6.0-SNAPSHOT for your pull request. After all it's a SNAPSHOT release.


Failure information is limited in socket server

Feature Request
The socket server presently only remits a failure_information array with stack traces organized by cause. However, human-usable information, such as that raised by FastlaneError are not remitted or displayed on stdout or stderr, making it hard to help a user diagnose their problem.
Feature Description
Basically, we should at a minimum add the class and message method results of the StandardError that is the basis for sending the failure payload in the socket server. These two in combination would allow a client to make the kinds of decisions that the UI class makes for remitting information to the user.
The only canonical socket interface for Fastlane is Swift, which uses payload as a dictionary. As long as we extend the shape rather than changing already-used keys, this will not break anything.
Alternatives or Workarounds
Since this is a matter of information not coming over the transom, there is no workaround - can't hack what you don't have.
I am putting together a PR to solve this issue.

On the TCP level the tuple (source ip, source port, destination ip, destination port) must be unique for each simultaneous connection. That means a single client cannot open more than 65535 simultaneous connections to a server. But a server can (theoretically) serve 65535 simultaneous connections per client.


megalinter github workflow

Motivation and Context

What has changed

How to test?

Links

Screenshots (if appropriate):

spike no longer needed


'Unexpected error encountered' when adding user without group selected

Version: master.20140722140215_bb27493
To reproduce:

Create a user
Fill in userid, name, password, email
Do NOT select a group
Click Add

Result:
MiqGroup(#24942120) expected, got NilClass(#19953760) [ops/rbac_user_edit]
Add button should be disabled if Group not selected.

@weissjeffm it appears this issue is now resolved. If so, please close this issue. Thanks


Removed unused namespaces and parameters

Removed unused private bool _thumbClicked from DarkUI/Controls/DarkScrollBar.cs
Removed unused namespaces
Properly formatted whole code

Go farm your metrics elsewhere


DOC-246: Documenting YAML config options for PerconaFT

This has the wrong base branch. Here is the correct PR for 3.0
https://github.com/percona/psmdb-docs/pull/35


Windows: Fix Isolation

Signed-off-by: John Howard<EMAIL_ADDRESS>Fixes #18077
This revendors engine-api to include the fix for https://github.com/docker/engine-api/pull/73 (@calavera) and completes the fix for https://github.com/docker/docker/issues/18077

LGTM

LGTM

LGTM


[web] load custom font in css file is not woking

font file is loacted in 'src/fonts/fontname.woff'
css
@font-face{
    font-family: 'fontfamily';
    src: url('./fonts/fontname.woff') format('woff');
}

but browser cannot load woff because './asset/asset/randomhex.woff' is not found
where should i put woff file?

@whdgmawkd
just released the patch to support woff extensions. could you try to run this in<EMAIL_ADDRESS>what you need is to place your fontname.woff in ./appConfigs/base/fonts folder.
rnv takes care of the rest for you.
in your code you simply use it as standard font family:
const styles = StyleSheet.create({
    someComponent: {
        fontFamily: 'Your-Custom-Font-Name`,
    }
});

behind the scenes explanation:
rnv loads all font files in the folder:
packages/app/appConfigs/base/fonts
based on your renative.json prop you can configure:
"includedFonts": ["*"],

then generates:
packages/app/platformAssets/runtime/fontManager.web.js
and
packages/app/platformAssets/runtime/fonts.web.js


start the game over the server When we create a room, it opens as <IP_ADDRESS> instead of the server.

When we create a room, it opens as <IP_ADDRESS> instead of the server.
in short
I think the opened rooms should be hosted on the server so that the players do not have connection problems
Currently, when we start the room game, the server starts from our own computer.

Duplicate of https://github.com/L-Leite/cso2-master-server/issues/47


[rom_ctrl] Fix backdoor path

We assume u_reg is the instance name of reg top in RAL model.
Need to find a better way to handle multiple regfile in one IP.
This still works for one CSR regfile and one mem regfile in a IP, as we
treat the mem backdoor separately. But can't handle if there are 2 CSR
regfiles in a IP.
@rswarbrick can you advise how to know the instance name of reg top
when generating the register in RAL.
Signed-off-by: Weicai Yang<EMAIL_ADDRESS>

Ah! I hadn't thought about this! For all the other per-device-interface names, we have logic which adds the "iface_name" suffix if there is one. Maybe we should just update the RAL generation code to do the same? (This actually ends up keeping the code in rom_ctrl.sv unchanged, but that's pure fluke).
I'll push up a PR showing what I mean in a bit.

#5856 is a better fix. Close this one.


Bump elasticsearch and logstash along with misc fixes/cleanups

fixed fab test target so it defaults to local_docker settings
adjusted prompt for auth url
added stray indirect dependency on functool32
locked down requirement versions
Renamed full_install to install, and install to partial_install
Removed references to partial_install from RPM spec and INSTALL doc
since it is a VM packaging function (still documented in docstring).
fixed string escape in echo commands
removed logstash-contrib from install list and added translate plugin

fab test ok.
general run through of client ok.

:+1:

Fab test didn't work for me, but I'll bet it's local to my machine, from all the restarting I've done while working on the events API.
It's a permission failure, somewhere.
I'll debug it, but if you have an "ah HA!" reaction and know where I should look, let me know!

@stugots try doing 'docker-compose stop' then 'docker-compose up'.  There is a new postgres container that grants the goldstone user 'CREATE DATABASE'.

Ah HA!

On Jun 11, 2015, at 1:28 PM, John Stanford<EMAIL_ADDRESS>wrote:
@stugots try doing 'docker-compose stop' then 'docker-compose up'. There is a new postgres container that grants the goldstone user 'CREATE DATABASE'.
—
Reply to this email directly or view it on GitHub.

all comments addressed in latest commit.

:+1:


Jump to relative link to another markdown file broken in some instances

I can't figure out why, but in one of my notes directories, jumping to relative links to markdown files works, and in some it doesn't.
The only difference that I can see is that one links to a file in a subfolder of the parent directory. I tested this by adding another markdown file in the same directory as the main.md that's not allowing me to jump and linking it, and even when at the same level in the tree, it doesn't work.
Feature working properly
Screencast from 09-04-2024 08:42:06 AM.webm
.
├── future-stuff.md
├── links.md
├── meeting-notes.md
├── meetings
│   ├── 2024-08-29.md
│   └── 2024-09-01.md
├── misc.md
├── notes.md
└── todo.md

Feature not working
Screencast from 09-04-2024 08:48:41 AM.webm
.
├── CS420
│   ├── lectures
│   │   └── 1.md
│   └── main.md
├── CS444
│   ├── lectures
│   │   └── 1.md
│   └── main.md
├── CS460
│   ├── lectures
│   │   └── 1.md
│   └── main.md
├── main.md
├── other.md
└── PHYS113
    ├── discussions
    │   └── 1.md
    ├── lectures
    │   └── 1.md
    ├── main.md
    └── syllabus.pdf

I'm really stumped on this. Please let me know if anyone things of any other method of troubleshooting

This morning I realized it's because I didn't run git init or have a .marksman.toml file in the root directory. Should have read the readme, woops


Fix state column bug

It turns out that the custom column name was not being respected on update (i.e. via event_name!). This adds a failing test for that, and then fixes it.

Coverage remained the same at 93.333% when pulling 1d71268e0e826bd0d80b640c0ff09cbd4b293d58 on bgentry:fix-state-column-bug into 068ad13ec60147564f02ab4d0493763dcd758bbd on cnsa:master.

@bgentry , Thanks! Good job.

@bgentry Thanks, done!


RuntimeError

TypeError: Block.forward() got an unexpected keyword argument 'past_key_value'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/data/yxy/Mistral-Interact-master/./src/test_one_new.py", line 280, in
main()
File "/data/yxy/Mistral-Interact-master/./src/test_one_new.py", line 277, in main
main_loop(args, raw_tasks, tokenizer, model)
File "/data/yxy/Mistral-Interact-master/./src/test_one_new.py", line 206, in main_loop
print(pred)
^^^^
UnboundLocalError: cannot access local variable 'pred' where it is not associated with a value

Refer to https://github.com/OpenBMB/Tell_Me_More/issues/3


Update java sources to fix IE11 layout issues

This change is

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks broke.

Comments from Reviewable

Reviewed 3 of 3 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Reviewed 2 of 2 files at r3.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Vision API with Azure OpenAI doesn't return any usage details like tokens used

Confirm this is a Node library issue and not an underlying OpenAI API issue

[X] This is an issue with the Node library

Describe the bug
Not sure if this is only occurs with Azure OpenAI, but the response doesn't contain usage credits at all.
Example response:
{
  "choices": [
    {
      "content_filter_results": {
        "hate": {
          "filtered": false,
          "severity": "safe"
        },
        "self_harm": {
          "filtered": false,
          "severity": "safe"
        },
        "sexual": {
          "filtered": false,
          "severity": "safe"
        },
        "violence": {
          "filtered": false,
          "severity": "safe"
        }
      },
      "finish_reason": "stop",
      "index": 0,
      "message": {
        "content": "The response here",
        "role": "assistant"
      }
    }
  ],
  "created": 0,
  "id": "",
  "model": "",
  "object": "chat.completion",
  "prompt_filter_results": [
    {
      "prompt_index": 0,
      "content_filter_results": {
        "hate": {
          "filtered": false,
          "severity": "safe"
        },
        "self_harm": {
          "filtered": false,
          "severity": "safe"
        },
        "sexual": {
          "filtered": false,
          "severity": "safe"
        },
        "violence": {
          "filtered": false,
          "severity": "safe"
        }
      }
    }
  ]
}

To Reproduce
Use Vision API with Azure OpenAI Endpoint.
Code snippets
No response
OS
Windows 11
Node version
Node v20.11.0
Library version
openai v4.28.0

cc @kristapratico

Thanks for reporting!
This sounds like an issue with the underlying API and not the Node library, so I'm going to go ahead and close this issue.

@Ahmet-Dedeler I'm not able to reproduce this; I see usage details as long as I'm not streaming the response. Could you share more details about the model and API version you're using, and add an example of how you're calling the vision API?


Migrate HTTP module to interop (HTTP/1, HTTP/2 and websocket)

Fixes #19939, #20081 and #20126
Check List

[x] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

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

@@            Coverage Diff            @@
##             master   #20106   +/-   ##
=========================================
  Coverage          ?   19.79%
=========================================
  Files             ?       55
  Lines             ?     1470
  Branches          ?      227
=========================================
  Hits              ?      291
  Misses            ?     1163
  Partials          ?       16

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 7dc0fb9...6738ac6. Read the comment docs.


When I run train.py, I got an import Error

Traceback (most recent call last):
File "train.py", line 18, in
import commentjson
File "C:\Users\hydraz320\Anaconda3\lib\site-packages\commentjson_init_.py", line 1, in
from commentjson import dump
ImportError: cannot import name 'dump'
May I ask that which os you were using?  Thanks a lot.

This seams to be an issue with your commentjson version. I am using Vesion commentjson==0.6.
A work around would be to replace from commentjson import dump with from json import dump. You can't use comments in the config file then. However KittiSeg currently does not have any comments in it, so it will work out of the box.

Hi, I am using commentjson 0.6 as well, but I get the same error. It might be windows (although I'm on Debian 4.9.13-1) and/or python>3.4 related, see https://github.com/vaidik/commentjson/issues/3.
I have used your workaround and it does not complain anymore, but it might have led to a different problem. I'll submit a separate issue for that in case it's unrelated.

@WPSanberg
Hello, I've been getting the same error, then it was suggested in the Issue 8 to use Python 2.7, I installed everything for 2.7 and the issue was resolved for me.

Well, I use python2.7 and ubuntu16.04 and get the expected results.

ok. This issue seams to be fixed. So I close it for now.
Regarding commentjson: I properly should look for an alternative if it does not work with python 3.4. In practise this is a minor think. In theory json could be used and/or any wrapper which removes comments.


node --instance  does not produce N number of ipcs

Describe the bug
I'm trying to start node and communicate with it via ipc.
Because of performance reasons I would like to spread the load across multiple files.
As described in the topic I have an issue with accomplishing it.
I must admit node is not in sync.
Steps to reproduce
Execute:
reth node --instance 3

Produces:
ls /tmp
reth.ipc-3

While I would expect
ls /tmp
reth.ipc-1
reth.ipc-2
reth.ipc-3

Node logs
No response
Platform(s)
Linux (x86)
What version/commit are you on?
reth Version: 1.0.5
Commit SHA: 603e39ab74509e0863fc023461a4c760fb2126d1
Build Timestamp: 2024-08-12T14:15:11.150189859Z
Build Features: asm_keccak,jemalloc
Build Profile: maxperf
What database version are you on?
2024-09-18T20:07:37.505596Z  INFO Initialized tracing, debug log directory: /root/.cache/reth/logs/mainnet
Error: Datadir does not exist: "/root/.local/share/reth/mainnet"
Location:
/project/crates/cli/commands/src/db/mod.rs:71:9
Which chain / network are you on?
tried with main, but seems it is not related to the issue
What type of node are you running?
Archive (default)
What prune config do you use, if any?
No response
If you've built Reth from source, provide the full command you used
No response
Code of Conduct

[X] I agree to follow the Code of Conduct

This is not the intended behavior for the --instance flag. It does not produce N IPCs or RPCs, it just changes the IPC path / RPC endpoint so it doesn't conflict with another reth instance on the same machine.
For example you would run
reth --instance 1
reth --instance 2

And it would run two reth nodes, with two IPC paths. If you have a particular application that is currently saturating IPC throughput and is now bottlenecked by IPC, then it would be interesting to know what that application is, or how we could support it. But that would be another issue for a different feature.


angular4 webpack2 build error

feature request:
polyfills.0836852….js:2 Unhandled Promise rejection: Template parse errors:
Can't bind to 'mdmenutriggerfor' since it isn't a known property of 'a'. ("Home Blog About ][mdmenutriggerfor]=theme>Theme <md-menu #theme=mdMenu>  <md-icon class=text-"): ng:///AppModule/AppComponent.html@0:152
Can't bind to 'mdmenutriggerfor' since it isn't a known property of 'button'. ("dashboard Blue Ocean   <button class=pull-right md-button [ERROR ->][mdmenutriggerfor]=account>TaylorPzreal <md-menu #account=mdMenu>  <md-"): ng:///AppModule/AppComponent.html@0:567 ; Zone:  ; Task: Promise.then ; Value: Error: Template parse errors:
Can't bind to 'mdmenutriggerfor' since it isn't a known property of 'a'. ("Home Blog About ][mdmenutriggerfor]=theme>Theme <md-menu #theme=mdMenu>  <md-icon class=text-"): ng:///AppModule/AppComponent.html@0:152
Can't bind to 'mdmenutriggerfor' since it isn't a known property of 'button'. ("dashboard Blue Ocean   <button class=pull-right md-button [ERROR ->][mdmenutriggerfor]=account>TaylorPzreal <md-menu #account=mdMenu>  <md-"): ng:///AppModule/AppComponent.html@0:567
at syntaxError (http://localhost:9000/dist/vendor.08368524123563a175f8.js:2:293382) []
at TemplateParser.parse (http://localhost:9000/dist/vendor.08368524123563a175f8.js:2:484994) []
when i use webpack in my dev it's success, but when i build it get those errors.
my development:
Linux + Chrome
Angular4.0.2 + @angular/material2.0.0-beta.3 + webpack2.3.3

At first glance it looks like something minifies your HTML in a non-case-sensitive way. This causes the directive to be not valid anymore.
Are you using the HTML-Loader? If so, with some special configuration?

@DevVersion thank you very much, i used html-loader, when i changed to 'raw-loader', get success.

change 'html-loader' with 'raw-loader' can fix the bug;

 new webpack.LoaderOptionsPlugin({
   htmlLoader: {
     minimize: DEVELOPMENT // workaround for ng2
   },

###       // minimize: PRODUCTION, // Comment out this line
debug: DEVELOPMENT,
options: {
context: __dirname
}
})

same error with karma unit-tesing, solution hereA "Testing with Material Components" guide to demonstrate including core theme in test suite #4056


big PR to enhance docker setup

Sorry for a single big PR

use multi-stage build to save 13MB on final image size
change auth.sh to entrypoint script
enhance entrypoint to look for either bind-mount of .sfdx, then envvar, then fail if neither exists
add compose file for docker-compose run sfdx option

Thanks @BretFisher. Got a cheat sheet of all the commands you ran? 😁 If not, no worries -- I can review the recording.

added .dockerignore

ah man my mic was distorted on live stream.. macOS moved it to 100% :(

@BretFisher Sounds fine to me. Watching right now.

here's a shell history, a total mess but answers are in there somewhre :) https://d.pr/n/L1U2YY/oLi7vWLYzC


gup: new package

Tested on nixpkgs unstable.
Since I'm both the maintainer and upstream author, I wanted to reuse the .nix expressions from upstream, so I'm importing them directly. There was some recent discussion on the mailing list encouraging this (which is the only reason I realised this was seen possible), but let me know if there's anything wrong with it.
The upstream nix expression is here, if you want to see that too: https://github.com/gfxmonk/gup/blob/f185052e2177ed5e46720e6c6cfb529b96cd17e2/nix/gup-python.nix

Hmm, looks like that didn't work (travis failed) because nix-env -q doesn't have enough metadata without downloading and importing the upstream nix expression. Any advice?

I've force-pushed a new version, which wraps the upstream derivation via a symlink, which should make the actual package evaluate fine for nix-env -q now. I've now split it into two commits; the first is to introduce a new wrapper function in build-support for wrapping external / upstream derivations.

Hmm, looks like that still doesn't work - it got past nix-env, but now it's unable to calculate nix-env -qaP --drv-path without fully evaluating all the attributes of the package. Is there anything I can do, or is it just not possible to have any part of a derivation be fetched externally?

OK, third attempt. I've abandoned trying to import the upstream nix expression, and now I'm just including a copy of it.

Please rename commit's name following our guide in CONTRIBUTING.md.

@jagajaga oops, renamed now.


Missing interrupts in atsam4s svd files

Found some more issues with the svd files. Seems like some interrupts are missing.
SUPC (Supply Controller) - 0
RSTC (Reset Controller) - 1
RTC (Real Time Clock) - 2
RTT (Real Time Timer) - 3
WDT (Watchdog Timer) - 4
      ...
   <peripheral>
      <name>EFC0</name>
      <version>6450M</version>
      <description>Embedded Flash Controller 0</description>
      <groupName>EFC</groupName>
      <prependToName>EFC0_</prependToName>
      <baseAddress>0x400E0A00</baseAddress>
      <addressBlock>
        <offset>0</offset>
        <size>0x200</size>
        <usage>registers</usage>
      </addressBlock>
      <interrupt>
        <name>EFC0</name>
        <value>6</value>
      </interrupt>
      <registers>
     ...

vs.
      ...
    <peripheral>
      <name>RTT</name>
      <version>6081J</version>
      <description>Real-time Timer</description>
      <groupName>SYSC</groupName>
      <prependToName>RTT_</prependToName>
      <baseAddress>0x400E1430</baseAddress>
      <addressBlock>
        <offset>0</offset>
        <size>0x200</size>
        <usage>registers</usage>
      </addressBlock>
      <registers>
      ...

Turns out they are missing for atsam4e as well, working on a fix.

Resolved with https://github.com/atsam-rs/atsam-pac/pull/14


Investigate 1.75.0-insider slowdown on perf bot

The perf bot shows a slowdown when we changed from 1.74.0 to 1.75.0. We should figure out if this is a fluke or some unwanted changed that happened during debt-week

The perf machine data shows a slowdown when processing configuration defaults and just configurations. That would explain a little bit but not all of the slowdown

@sandy081 Were there changes in the configuration service (during the 1.74 to 1.75 debt week) that could explain this or should I fish for a new listener here?

No changes in config land except this which is adding log statements.

Good news: great improvements wrt handling contribution points.

Bad news: something happens with added a big delay between the didCreateWorkbenchContributions/3 and willHandleExtensionPoints phases

Caused by https://github.com/microsoft/vscode/commit/e90f48f1a9d58f257b8fd836b5f0a2b9b8e9d110#diff-72bc8a9f7335192506e81a38f38cec5f7438af42467722f97f1f57e58be2a372R516
Fix - https://github.com/microsoft/vscode/pull/170446

I think this is solved:

Confirmative - four fast builds in a row don't lie!

Thanks everyone for helping out on this!


Add docs for _custom_id_ for lambda function URLs

This PR documents the changes made in this localstack-core PR.

Marking the PR as draft since the feature isn't implemented yet.


How to install python plugins?

One of the plugins is created using Go and I've got it installed, but I can't figure out how to get the Python based plugins installed. I've used Python before and know how to use pip, but I can't seem to figure out how to install this. Using e.g. confluent plugin install confluent-api_key-purge doesn't work, because it's unable to find the python file for the plugin. Even when you download the python file locally and point confluent to it confluent plugin install .\confluent-api_key-purge.py, it doesn't work.
I'm using Python 3.9.
UPDATE:
OK, after running confluent plugin -h, it looks like the confluent CLI expects the plugins to be an executable. The Python plugins are just .py files. I ended up using ChatGPT to help me convert/compile the .py into a .exe, by doing this:
Step 1: Install PyInstaller
First, you need to install pyinstaller. Open a terminal (command prompt) and use pip to install it:
pip install pyinstaller
Step 2: Navigate to your script's directory
Using the terminal (command prompt), change the directory to the location where your Python script is located. For example:
cd path/to/your/script/directory
Step 3: Create the .exe file
Run the pyinstaller command to create the .exe file. Replace your_script.py with the actual filename of your Python script:
pyinstaller --onefile your_script.py
The --onefile option bundles everything into a single .exe file. If you don't use this option, pyinstaller will create a build with multiple files, which might include a bunch of supporting files and a folder containing the executable.

Hi, can you post the terminal output when you run confluent plugin install confluent-api_key-purge? Also, what operating system are you using?

Hi, can you post the terminal output when you run confluent plugin install confluent-api_key-purge? Also, what operating system are you using?
Edit: I'm assuming you're on Windows since you compiled it into an exe?
Edit 2: If on Windows, can you post your PATHEXT environment variable?

Yes, I'm on Windows 11.

I imagine that for every extension that isn't available in PATHEXT, we'd have to add it there?

Unfortunately, that's currently the case on Windows. The plugin functionality uses PATHEXT to filter out non-executable files.
I'll see what we can do to improve this. In the short term, though, it will probably just be improvements to the CLI messaging.

Interesting. I know about PATH but never used PATHEXT before. I would have thought that it'd check if Python binary is in PATH (which it is as per my screenshot) and just call Python to run some pip install command or something.
Anyway, good to know.


Fixes to Non-Canonicalized Internal Links

These are absolute URL fixes to issues we are encountering with non-canonical internal links due to relative link conventions.
We are contracted for SEO work with Prisma - we typically work with Etel, Alex, Justin, and Daniel.
Describe this PR

Changes

What issue does this fix?

Any other relevant information

Hi @janpio,
Resulting from our routine technical audits & site crawls, these particular internal links are, for some reason, defaulting to the non-canonicalized URL conventions. This is a serious issue for SEO as it confuses search engines as to what the true canonical version of a page is.
Additionally, just as a rule of best practice, absolute links are always recommended for SEO.

Can you please list the actual problems? We definitely want to fix any problems with links that exist.
But we do not want to inconsistently change random link targets in the source of our documentation as this really makes the writer and development experience worse.

Additionally, just as a rule of best practice, absolute links are always recommended for SEO.

I strongly disagree with this statement. Internal links are perfectly fine as relative links.

I had a quick look at the suggested changes:

As commented the ButtonLink seems to have a bug that I reported in https://github.com/prisma/docs/issues/2500 so it can be fixed
The removal of trailing slashes in links with # look valid and correct to me (although many of these could be turned into internal links
Most other changes seem to be links that do not follow the pattern of using fully qualified internal links (basically: Absolute link minus domain and /docs). That are oversights of a previous maintainers and also just bugs to be fixed.

Hi Jan,
Based on the above, it sounds like moving forward when we make these changes and all out that an internal link is using a non canonicalized version, should we be replacing with "fully qualified" internal links? It sounds like this then would fix our issue, and remove the need to use absolute links?
Here's an example: if you navigate to this page: https://www.prisma.io/docs/concepts/components/prisma-client/crud and find the anchor text "Working with relations > Nested writes" you will see that it is linking to a non-canonicalized page (https://www.prisma.io/docs/concepts/components/prisma-client/relation-queries/#nested-writes) whereas the correct version is https://www.prisma.io/docs/concepts/components/prisma-client/relation-queries#nested-writes
For that link, I see that this is being used as the rel internal: relation-queries#nested-writes
To fix, are you suggesting that we should simply change to /concepts/components/prisma-client/relation-queries#nested-writes instead of the absolute?
Looping in @imchairmanm as well since I'm assuming this would apply to /dataguide too

Yes, fundamentally that is the kind of link we usually want to have and that should be rendered correctly without the trailing slash.
(The example you chose is special in that it also has an anchor added to the link (#nested-writes), we might have a bug on that logic that always adds a trailing slash for example. But we would figure that out and then fix this independently)

Thank you.
Many of the issues discovered in our crawl contain anchors (we call those deep links) - should we wait until we hear more from you regarding whether there's a global bug related to anchor logic before beginning to correct those (in the event that for those, we might have to use absolute links)?

We would prefer to identify the bug and fix it. A simple way to see would be to change one of the absolute links here in your PR to use a fully qualified relative one, and then check the preview deployment if the link presents correctly in code. (I can also take some time tomorrow and do that if you do not have the capacity)

Hi Jan,
Again, really appreciate the time you're taking to work with us on this.
The approach you outlined above sounds good. We can move forward from now on with testing in the preview deployment to see if the fully qualified rel link approach works. In cases where it doesn't, we can flag that in our comments.

I split out one class of errors (the links with anchors, deep links as you all them) into a separate PR quickly: https://github.com/prisma/docs/pull/2502 Reviews and confirmations the problems are resolved in the Netlify preview very welcome.

I think we fixed all of these independently, so can close this one.


Updated documentation for Deduplication

[ ] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes

Verification

Thanks for the feedback. Will do the required changes.

Seems like https://github.com/thanos-io/thanos/pull/5057 is a copy of this. Thus, closing this.


Call to undefined method PragmaRX\Tracker\Vendor\Laravel\Facade::onlineUsers()

I have follow the readme.md but I found this issue and still haven't found the solution. Any solution for this problem? I'm using Laravel 5.7 btw

@kevingio did you find a solution for your problem?


Fix signal linker removing exception

About the PR
Fix exception on map loading
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

Related to https://github.com/space-wizards/space-station-14/issues/12732 ?

Related to #12732 ?

I didn't find any null associated with the device linking in the map file. More likely there are invalid uid used in linking.

They got updated to invalid in one of the serialization updates.


Update/redbean

@timothygwebb This still fails without csrfp being loaded. Please don't commit anything to this branch, I plan to either revert it back to here or start fresh and after that figure out the actual issue with the failing tests, thank you.

@timothygwebb This still fails without csrfp being loaded. Please don't commit anything to this branch, I plan to either revert it back to here or start fresh and after that figure out the actual issue with the failing tests, thank you.

Thanks

Closing in favor of a locally patched version. I will cleanup this branch and we can use it in the future once everything is figured out and we no longer need a local hack


Mention extensions can be installed from a repository

I also noticed there is no mention of FORCE INSTALL, I know it involves versioning but that's about it, so I don't think I'm the right person to add this information, but I think it should be added.

Thanks, looks great!


Retrieve value from DocValues in a flat_object filed

Description
Added ability to retrieve value from DocValues in a flat_object filed
Related Issues
Resolves #16742

Check List

[x] Functionality includes testing.
[x] API changes companion pull request created, if applicable.
[x] Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@kkewwei -- I plan to review this PR this week. Please @ me or bug me on OpenSearch Slack if I don't!

@kkewwei -- I plan to review this PR this week. (Lots of meetings this week, but should have time Thursday or Friday.) Please @ me or bug me on OpenSearch Slack if I don't!

Thank you very much, some of the unit tests are not ready, I will fix soon and at you when all is ready.

❌ Gradle check result for 54cd837: FAILURE
Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

SearchTimeoutIT.testSimpleTimeout #16056

❕ Gradle check result for a41d71e: UNSTABLE
Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure.

SearchRestCancellationIT.testAutomaticCancellationDuringFetchPhase #14311

OpenSearch

@msfroh please have a look when you are free.

@msfroh, please help to merge the pr.


add lodestar

Solves https://github.com/ethereum/staking-launchpad/issues/430  and https://github.com/ChainSafe/lodestar/issues/3008

Thanks for the review @CarlBeek . I've updated the descriptions based on your feedback.
One thing that could be done, is to make it configurable via an env var (e.g. REACT_APP_CLIENTS_LODESTAR_ENABLED).
That way we could have different configurations for each envionment. Mainnet would have that set to 'false', but Prater could have it set to 'true'.
What do you think?

I definitely like the direction this is going, definitely keen to get Lodestar up there.
Having had some internal discussions, we've decided to hold off on merging this for a while until we are absolutely convinced Lodestar is mainnet ready from a reliability & safety standpoint. By including a client on the launchpad, we are effectively endorsing them as a not-too-risky choice, and I'm just not prepared to make that determination yet.
Let's see how the new Prater Lodestar nodes do along with the mainnet ones (there are 2 I am aware of) and circle back after a while.

Hey @CarlBeek , thanks for this. We can keep you updated and discuss what other blockers to remove and milestones we can try to achieve before you decide to merge this in. I'm on the ethereum.org and ChainSafe discord servers as phil.knows#8609

Hi @CarlBeek, just an update on our side, we've implemented a production release track into Lodestar and have been running multiple mainnet validators through the Ethereum Client Incentive Program since mid-December. Feel free to reach out if you need any other specific milestones you'd like us to reach to merge this in. Thanks!

Hey @skylenet! Sounds like this is still temporarily on hold, so I just flipped this to draft mode for the time being. In the meantime I updated with the merge conflicts cleaned up.

Hey @CarlBeek @skylenet @wackerow, I just wanted to revisit this PR to get an idea of your perspective of what needs to be met for this to be merged. I believe that part of the reason why we're still struggling to see usage here is our lack of visibility in some important parts of the Ethereum ecosystem such as the staking launchpad. Although it is quite clear that people within the protocol community are well aware of 5 functioning consensus clients through our participation in major events such as the shadow forks of which we have had no large participation issues.
Some presentations at DevConnect consistently mention 5 consensus clients, including @djrtwo's presentation at EthStaker which included a screengrab of our logo because this PR was not merged. The Ethereum Foundation's report released on Apr 18 mentions 5 consensus clients ready for the merge and talks which included consensus clients always included Lodestar in some way, including the Secureum TrustX Deep Dive with consensus client teams. In addition, we have an upcoming audit in preparation of our v1.0.0 release scheduled this summer.
I would be happy to provide you with some metrics of our client on mainnet and discuss further in detail if there are any additional hesitations with pushing us up to production status & getting this merged.

Hello! The current status for this is that a third party audit on Lodestar is starting this week, and this will be completed towards the mid-end of July. Once this has been completed, the intention is to add Lodestar on the same locations as the other clients on ethereum.org and the launchpad.

Any update on this?

Hey @SeaMonkey82 , we're currently implementing fixes from the audit. They will get implemented in a future release and we'll do an announcement upon its completion. Shortly after that, we will give some time for users to upgrade and Lodestar will get listed, which includes this PR.

Hey<EMAIL_ADDRESS>any update here?

Hey @wackerow, we're targeting a v1.4.0 release for EF endorsement which would upgrade some of the dependencies for fixes to currently disclosed public vulnerabilities alongside the audit fixes. We decided to take a more conservative approach to address all publicly open eth client vulnerabilities before getting listed on the launchpad. Targeting Q1 2023 ideally before withdrawals are enabled. 🤞

Hey everyone, quick update here our own status with closing up security vulnerabilities and the audit. Our next release v1.4.0-rc.0  is in release candidate testing phase and will include all the fixes to satisfy the audit and other known vulnerabilities.
Hoping we can merge this very soon!

Awesome, thanks @philknows for the update!
Noting that this PR ended up being duplicated on #541, which appears to have branched off this and fixed a link or two.
I'm going to close this PR in favor of the updated one, please do me a favor and comment on #541 when this is ready 🙏  Thanks!


version 0.7.13 not published to npm

@zackbloom can you do this? I would but I don't have permissions to publish to npm for this project

@rawls238 thanks for the quick response.

I was going to open another issue regarding not publishing the NPM and found this issue instead :(
Currently as a workaround I have specified the git tag at package.json, but I think it will be nice if in the long run each every minor/major versions get published on NPM too.

it's been quite some time? Anyone can help with this?

Done, sorry for the delay!


Not-in filter excludes too many results

Which version of the client library are you using?
6.0.0
Which version of .NET are you using?
.Net Standard 2.1
Which operating system (Linux, Windows, ...) and version?
Windows
Actions taken
string query = $@"SELECT
campaign.id,
campaign.name,
campaign.serving_status,
campaign.status,
campaign.advertising_channel_type,
campaign.advertising_channel_sub_type,
metrics.absolute_top_impression_percentage,
metrics.average_cpc,
metrics.clicks,
metrics.conversions,
metrics.conversions_value,
metrics.cost_micros,
metrics.cost_per_conversion,
metrics.ctr,
metrics.impressions,
metrics.top_impression_percentage
FROM
campaign
WHERE campaign.advertising_channel_sub_type NOT IN ('DISPLAY_EXPRESS', 'SEARCH_EXPRESS')";
Expected result
A list of campaigns whose advertising channel type is not display_express and is not search_express
Actual result
A list of campaigns whose advertising channel type is neither display_express, nor search_express nor unspecified
From what I understand, the "not in" also filters the campaigns that do not have an advertising channel sub type

Hi Moerland,
This seems an issue with the underlying API. Could you please drop an email to<EMAIL_ADDRESS>with the following details?

The customer ID for which you saw the incorrect behaviour.
The API call logs. Instructions to turn on logging is here: https://developers.google.com/google-ads/api/docs/client-libs/dotnet/logging.
Specific campaign IDs / names you were expecting to see in the response, but didn't.

Feel free to add Attn: Anash to the email and reference this issue.
I'll close this issue here for now.


Refactor and simplify signal handling logic

This PR refactors some of the internal signal handling logic. It avoids a number of circular references to avoid relying on GC collecting cycles. It does not affect the public API, this is internal only.
Builds on top of #104 and #149

Removed WIP marker, this is now ready for review :shipit:


Refactor: Constants for Json Web Key

RFC 7517 constants from jose library

A suggestion too: instead of having a commit message saying "more refactorings", you could give just a tiny bit of extra information. For example "refactor: use constants" would be my choice for a commit like this one

This looks good to me.


Build fails for wasm32-wasi target: _Float16 is not supported on this target

Hello. As you may know, Mozilla bundles an older version of simde with their sources, and it works. I'm trying to use the latest version (0.8.2) installed by the FreeBSD port on my machine.
Unfortunately, compiling for the wasm32-wasi fails:
clang++18 --target=wasm32-wasi --sysroot=/opt/share/wasi-sysroot -o sse_optimized.wasm -c -I/n/home/ports/www/firefox/work/firefox-126.0.2/media/libsoundtouch/src -I/n/home/ports/www/firefox/work/.build/media/libsoundtouch/src -I/n/home/ports/www/firefox/work/.build/security/rlbox -I/n/home/ports/www/firefox/work/.build/dist/include -I/opt/include/nspr -I/opt/include -I/opt/include/nss -I/opt/include/nspr -I/n/home/ports/www/firefox/work/.build/dist/include/nss -I/opt/include -I/opt/include/libpng16 -I/opt/include/pixman-1 -Os -fno-exceptions -fno-strict-aliasing -DNDEBUG=1 -DTRIMMED=1 -DMOZ_IN_WASM_SANDBOX -DMOZ_SAMPLE_TYPE_FLOAT32=1 -DSOUNDTOUCH_ALLOW_SSE=1 -DSOUNDTOUCH_WASM_SIMD=1 -DSIMDE_ENABLE_NATIVE_ALIASES=1 -DBUILDING_SOUNDTOUCH=1 -DST_NO_EXCEPTION_HANDLING=1 -msimd128 -I/n/home/ports/www/firefox/work/.build/dist/stl_wrappers -MD -MP -MF .deps/sse_optimized.wasm.pp   /n/home/ports/www/firefox/work/firefox-126.0.2/media/libsoundtouch/src/sse_optimized.cpp
In file included from /n/home/ports/www/firefox/work/firefox-126.0.2/media/libsoundtouch/src/sse_optimized.cpp:65:
In file included from /opt/include/simde/x86/avx2.h:33:
In file included from /opt/include/simde/x86/avx.h:28:
In file included from /opt/include/simde/x86/sse.h:34:
/opt/include/simde/x86/../simde-f16.h:98:11: error: _Float16 is not supported on this target
   98 |   typedef _Float16 simde_float16;
      |           ^
1 error generated.

Is this a bug? Perhaps, I can set the SIMDE_FLOAT16_API to something as a work-around?

Hello @UnitedMarsupials and thank you for opening this issue.
Yes, you can #define SIMDE_FLOAT16_API 1 before importing any SIMDe header to work around this. Or as a compiler command line option -DSIMDE_FLOAT16_API=1.
What flags are set by the wasm32-wasi target? It would be good to add a fix near https://github.com/simd-everywhere/simde/blob/e30e6ec44cdbc85c04d6cb1f26fb6d93f43cc5ea/simde/simde-f16.h#L71

Yes, you can #define SIMDE_FLOAT16_API 1

Thanks! Let me try that.

What flags are set by the wasm32-wasi target?

Using the trick from here, I conclude, that, in case of clang, at least, these are:
#define __wasi__ 1
#define __wasm 1
#define __wasm32 1
#define __wasm32__ 1
#define __wasm__ 1
#define __wasm_mutable_globals__ 1
#define __wasm_sign_ext__ 1

Yes, you can #define SIMDE_FLOAT16_API 1

Thanks! Let me try that.

Did that work?
How can we test SIMDE with WASI in our CI? We can make a PR to not use the _Float16 if __wasi__ is defined, but I'll want a CI build & run of the tests under WASI to confirm


Develop - ROCm 5.7 Updates

rocAL Updates
Docs Updates
header updates

@LakshmiKumar23 can you verify this and approve for merge.


Linter sometimes fails to report issues on a specific PC

So for example

I create a few commits, push a PR
Build is executed on TeamCity, check task fails
Same task on my PC somehow ignores these errors
https://github.com/Kotlin/dataframe/pull/858
clean doesn't seem to do anything too

Maybe there's an issue that check doesn't depend on ktlintKotlinScriptCheck

Mayb

Maybe another task could be run. TC runs build right? So maybe you could run that one, or one specific to ktlint, like ktlintCheck (if I'm correct)


Link to docs in case of unkown command

There should be a page documenting the commands that it understands.

@charlesmike doesn't know many commands

@amihaiemil, sorry, I do not understand you. See this link for more details.

@charlesmike hello there!

Hi, @amihaiemil! I can help you index your Github-hosted website, provided that the repository name respects the format author.github.io and the command is given by the repo's author.

@charlesmike, hi!!!

Hi, @amihaiemil! I can help you index your Github-hosted website, provided that the repository name respects the format author.github.io and the command is given by the repo's author.

@charlesmike hello there!!

@charlesmike hi

dine


Implement ping/pong messages parsing and processing

We don't currently support ping/pong functionality.

BTW, C# guys now really do use pings for something real: neo-project/neo#1024


Add field "last updated" to memberslist

Fixes MA-116

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


text split options for "." still not splitting

I just had a chance to pull latest and revert my code to set the split array manually - but the original problem is back - not splitting on period. Am I not making the call correct?
d3plus.textwrap()
.container(nodeText)
.x(leftSideOffset)
.width(textLabelWidth)
.resize(false)
.padding(0)
.text({ value: label, split: ["-", "/", ";", ":", "&", "."] }) /// Added "."
.draw();

Closed with #427


Outgoing raid are treated as incomming raid

Improve condition check of EventSub raid message so it detect correctly if raid is incomming or outgoing


Fix #1604

@dreamsxin Could you please send this patch to the development branch?

@sergeyklay @AlexNDRmac
We can discuss the rules for adding prefixes, about containing or without points.

xx.xx

$namespace.'.'.$name;

xx

$namespace.'.'.$name;

@dreamsxin As I know @AlexNDRmac already done some work in this direction. Maybe he should do a PR on which we could work together.

I close that.


doc: release-notes: add GigaDevice changes

More GigaDevice support added, include drivers, boards, etc. Also, some known issue has been fixed.
Get GigaDevice change list:
$ git log -i --grep="gd32" v3.1.0..v3.2.0-rc2

Hi @cameled ,
I would like suggest to try always keep the release version at commit message for easy tracking.
For instance:
release-notes-3.2: Add GigaDevice changes

use doc: release: 3.2:, I found some commit already use this pattern, follow it.

@cameled Can you help resolve the conflicts.


Added MID0900 support

Adding MID 0900 support is a little bit tricky. The difficult part comes from the fact that a MID 0900 response comes back as a mixture of ASCII and binary data. Unfortunately trying to delimit with a null terminator ('/0') is not sufficient since parts of the binary data also use null terminators.
Hence my solution is to read the first 4 bytes to get the length of the entire message and break apart the ASCII and binary sections.
Once we have our ASCII length, we can parse the fields like normal. The binary section then needs to be converted into shorts and stored. I store both the raw and converted trace samples.
One last caveat I discovered was that in testing, I noticed that I never got the first NumberOfPIDDataFields. Reading the ASCII data, it always indicates the length as 0, meaning nothing is there. However the second NumberOfPIDDataFields has a length and therefore was populated with PID data.
I differentiated this by created a NumberOfPIDDataFields and a NumberOfPIDDataFields2.
Lastly, see my closed issue which explains that a specific firmware version is needed on the controller for it to receive/send MID 0900.

Hello @Josep313, first of all, thanks for the contribution!
If I'm not mistaken, Mid 0061 has the same kind of mixture, you can take a look if you want to check if there is another easier way to do it. But as far as I can see from your code, Mid 0900 is very strange hahaha
Just one more thing, can you add Unit tests to it?
With it we can assure the working feature and also, of course, test it.

No problem! I can start adding the unit tests when I have some free time, as well as look over MID 0061 to see if there's an easier way to parse ASCII + binary data.

Hello @Josep313, first of all, thanks for the contribution!
If I'm not mistaken, Mid 0061 has the same kind of mixture, you can take a look if you want to check if there is another easier way to do it. But as far as I can see from your code, Mid 0900 is very strange hahaha
Just one more thing, can you add Unit tests to it?
With it we can assure the working feature and also, of course, test it.

No problem! I can start adding the unit tests when I have some free time, as well as look over MID 0061 to see if there's an easier way to parse ASCII + binary data.


In electron history.push

Version "react-router": "^4.3.1",
This only happens from context menus and the main menu in electron. Route goes to target component then calls componentDidMount in the previous component then renders target component without calling componentWillUnmount of any of the involved routes.

This is a bug tracker, not a support system. For usage questions, please use Stack Overflow or Reactiflux where there are a lot more people ready to help you out. Thanks!