fix error on machines with pyspark installed where passed dataframe is not spark pandas

Description
PM ran into an error when using latest erroranalysis package on a machine with pyspark installed (Compute Instance), but where they were not passing in a spark pandas dataframe.
Strangely, the error was related to the import:
AttributeError: module 'pyspark' has no attribute 'pandas'

Importing import pypark worked, but using pyspark.pandas.frame.DataFrame did not - but it started working after explicitly importing import pyspark.pandas as ps.  Note on older versions of pyspark, the namespace pyspark.pandas may not even exist though, so I added try/except just to be very safe and ensure this never causes issues.
Checklist

[ ] I have added screenshots above for all UI changes.
[ ] Documentation was updated if it was needed.
[x] New tests were added or changes were manually verified.

Codecov Report

Merging #1415 (fefc640) into main (9441f9b) will decrease coverage by 5.51%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1415      +/-   ##
==========================================
- Coverage   85.95%   80.43%   -5.52%
==========================================
  Files         100       18      -82
  Lines        4798      322    -4476
==========================================
- Hits         4124      259    -3865
+ Misses        674       63     -611

Flag
Coverage Δ

unittests
80.43% <ø> (-5.52%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

erroranalysis/erroranalysis/_internal/utils.py

erroranalysis/erroranalysis/version.py

...ai/responsibleai/_tools/causal/causal_constants.py

responsibleai/responsibleai/__init__.py

responsibleai/responsibleai/_input_processing.py

...i/responsibleai/managers/error_analysis_manager.py

...ponsibleai/responsibleai/rai_insights/constants.py

raiwidgets/raiwidgets/error_handling.py

...tools/error_analysis/dashboard_schemas/__init__.py

...bleai/responsibleai/_tools/causal/causal_result.py

... and 72 more

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 9441f9b...fefc640. Read the comment docs.

@gaugup yes, we do have a test that I added:
https://github.com/microsoft/responsible-ai-toolbox/blob/main/erroranalysis/tests/test_pyspark_surrogate_error_tree.py
I've only run it locally, it's not yet setup in a gate.  I was planning on eventually adding a new github action to test this using:
https://github.com/marketplace/actions/setup-apache-spark
It's quite complicated to setup spark on a machine locally so I think having a separate github action for spark tests would be the best way forward.

this seems to be kind of a weird issue which happens when spark is installed, but the dataframe passed in is a pandas dataframe (as usual), or that's what I've understood from when I was able to reproduce it on a compute instance


Draft MapLibre Strategy 2023 - 2026

Write a two-page draft of the MapLibre Strategy 2023 - 2026. Focus on dream and high-level goals.

https://www.hotosm.org/strategic-plan as inspiration

Became a 6 page draft. In review at the Governing Board until Monday Feb 13th, 2023.


Hi! some questions about dropblock.

Hi! During training for ResNest, the dropblock is used to improve the accuracy, but is dropblock really useful? Have you done comparative experiments and how much drop_prob is in training?
Thank you!!!

Using Dropblock can slightly improve the performance ~0.2% for ResNeSt-50 (sometimes the improvement is not consistent).

Hi！ can you tell me the drop rate for dropblock during training ?
Thank you !

The drop rate is linearly increased from 0 to 0.1.


webpack -d devtool default is not overridden in config

From the docs it states:
"Note: With the next major version the default for -d will change to cheap-module-eval-source-map"
The behavior I'm seeing is the devtool config is ignored if I run webpack -d, it is honored if I just run webpack. The docs make it sound like cheap-module-eval-source-map is just a default and I would expect the config to be the value used if it is set. This was happening on both beta 4 and 5.

CLI options override the configuration options. So -d overrides devtool: ....
--devtool eval overrides -d.

This issue was moved to webpack/webpack.io#118


Switch Quilt and Purpur to image helper

Also:

normalize all VERSION processing including using mc-image-helper
Resolves #2227

Pulls in

https://github.com/itzg/mc-image-helper/pull/245
https://github.com/itzg/mc-image-helper/pull/247
https://github.com/itzg/mc-image-helper/pull/248

[init] ERROR failed to install Modrinth modpack
[init] Running as uid=0 gid=0 with /data as 'drwxrwxr-x+ 1 0 0 1742 Jul 14 15:20 /data'
[mc-image-helper] 15:39:08.323 ERROR : Invalid parameter provided for 'install-modrinth-modpack' command: Unable to locate requested project given ProjectRef(idOrSlug=hermitcreateflavors, versionType=null, versionId=null, versionName=1.0.1)

config file env
JAVA_VERSION=jdk-17.0.7+7
VERSION=LATEST
MODRINTH_PROJECT=https://modrinth.com/modpack/hermitcreateflavors/version/1.0.1
MOD_PLATFORM=MODRINTH
EULA=TRUE
TYPE=FORGE

errors out.. - why

@SirBastol next time, please create an issue rather than commenting on a random PR. In any case, that project page doesn't exist:

It was the google result. Of the search of that error.
The page exists but is private.
Regardless forge server. Doesn’t launch
Something about a zip error  that makes no sense. When set as forge and
manually uploaded mod files in mod dir. there are no zips
On Fri, Jul 14, 2023 at 7:33 PM Geoff Bourne @.***>
wrote:

@SirBastol https://github.com/SirBastol next time, please create an
issue rather than commenting on a random PR. In any case, that project page
doesn't exist:
[image: image]
https://user-images.githubusercontent.com/988985/253711606-1d5d446c-fa49-4ac4-b6c9-1b1e1bb8d6e5.png
—
Reply to this email directly, view it on GitHub
https://github.com/itzg/docker-minecraft-server/pull/2229#issuecomment-1636630794,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABQBDDMUNURSVOXDKJG7HD3XQH6PZANCNFSM6AAAAAAZKLIDEQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Thank you ,
Paul Cole
“Defeat is a state of mind. No one is ever defeated until defeat has been
accepted as reality. "
- Bruce Lee


Wireshark is opened before connecting to the named pipe

This PR replaces the PR #11 I guess

Yes, we could have reused #11 but no biggy!


Show what sorts of errors are thrown when trying to delete containers with immutable storage

I think that it would be nice to show customers what sort of errors they may encounter when trying to delete containers where immutable policy is set and what the reasons may be. Most of them are already mentioned on this page but it is hard to understand exactly what the cause is because the error is sort of generic.
For example, if a customer is trying to delete a container, they may get the following error:

This lets them know that immutable storage with versioning is enabled and part of the issue but not what exactly to do. It may be that there is an access policy which is preventing this action. In the case I had today it looks like the issue will be that the container itself has blobs inside of it and therefore the container cannot be deleted until they are deleted first. We checked and found there is no policy on the container itself but that the version-level immutability support is turned on for the storage account, even though there are policies defined. On this page the documentation this scenario is laid out as such
"If version-level immutability support is enabled for a container and the container contains one or more blobs, then you must delete all blobs in the container before you can delete the container, even if there are no immutability policies in effect for the container or its blobs."
Maybe even just making this a note to bring more attention to it along with that you may get an error as shown above would be sufficient. That might also be nice for the same statement about deleting the storage account, itself.
It may also be good just to have a section devoted to the deletion of the storage account/container where immutable storage support is enabled, and either link to other sections in the page or keep all the data together in this section(this would make it easier to understand).

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

ID: cc9858b4-7142-677d-9c19-3c3b71aeeede
Version Independent ID: 68d4d252-4c5b-befb-1110-666d4a3e16af
Content: Configure immutability policies for blob versions - Azure Storage
Content Source: articles/storage/blobs/immutable-policy-configure-version-scope.md
Service: storage
Sub-service: blobs
GitHub Login: @normesta
Microsoft Alias: normesta

@jgross-njnc
Thanks for your feedback! We will investigate and update as appropriate.

@jgross-njnc
I'm going to assign this to the document author so they can take a look at it accordingly.

@normesta
Can you please check and add your comments on this doc update request as applicable.

Thanks for your contribution to our documentation
Unfortunately, we have been unable to review this issue in a timely manner. We sincerely apologize for the delayed response. We are closing this issue. If you feel that the problem persists, please respond to this issue with additional information.
?
Please continue to provide feedback about the documentation. We appreciate your contributions to our community.
#please-close


Read hooks call contract function with incomplete args resulting in console errors

Scenario
When using a contract read hook to call a function that requires some arguments, a common approach is to create a form to let a user provide his input. These form states are then used as params to pass into the args.
Current behavior
Currently, the hook is triggered on each variable change, resulting in a lot of errors in the console as long as not all args are set and of the correct type. To resolve this, you need to either set the hook to enabled: false and create some custom condition when to enable it based on params or trigger it with the refetch.
Proposed solution
I think a more intuitive behavior is to disable the hook when we know it would result in an error, and only call the contract function when the params are good. That way, the types force a dev to plug in correctly typed variables for args, and when a user using the UI does some action (i.e. fill in a form) that actually provides a value for the variables, only then is the actual contract function called.
Benefits

Reduce contract calls (or at least reduce triggers of the wagmi hook)
Reduce a lot of boilerplate code to prevent these unnecessary calls
Cleaner console output

Tysm @sverps  !! Even I noticed it while testing

I think a more intuitive behavior is to disable the hook when we know it would result in an error, and only call the contract function when the params are good.

Yup 💯, even wagmi's native hooks do the same thing, they don't make a request if args are undefined


Cannot declare class Redis

Untouched Laravel 5.5.12 + predis:

PHP Fatal error:  Cannot declare class Redis, because the name is already in use in _ide_helper.php on line 14394

I think the Redis class is already declared in Illuminate/Support/Facades/Redis.php

Are you including the file in your code?

This is a theoretical research with phpstan without any custom code.
I've installed laravel/laravel and added
"predis/predis": "~1.0",
"barryvdh/laravel-ide-helper": "dev-master"

After running artisan ide-helper:generate _ide_helper.php is created, then I've ran phpstan.

Then phpstan reported the duplicated class declaration.

Yeah it's not actually used code, so nothing to worry about.


Formate usage/tables documentation pages via markdownlint

Is your feature request related to a problem? Please describe.
Based on the discussion in #572 the documentation pages within the usage/tables directory should be formatted with markdownlint.
Describe the solution you would like
The following should be included:

Formatting of the usage/tables documentation pages, as described above.
A custom markdownlint configuration file to include formatting rules different from the default ones.
Minimal changes for the user-facing documentation, when needed.

Additional context
See #572 for the motivation.

take


Error

When I start mumble-voip I get this error: "@mumble-voip/server.lua:15: attempt to call a nil value (global 'MumbleCreateChannel')"

Are you on an old server version?

yeah, I'm planning on updating though, so it'll work when I update?

Do it and u will see.


Show disabled days?

Hello, is it possible to show disabled days, but greyed out and unclickable?
I have already tried disabling the day and adding a Span after that, but that doesn't work as disabling a day just hides that day completely.
I have also searched for Events or other methods to make the date unclickable and then adding a Span, but I didn't find anything like that.
Is there a way?

app:mcv_showOtherDates="all"

app:mcv_showOtherDates="all"

Thank you @venky1893  , its working

It's been a long time, but I was able to solve this issue somehow.
For reference, see here:
https://github.com/ThexXTURBOXx/studip-app-uni-passau/blob/master/app/src/main/java/studip_uni_passau/femtopedia/de/unipassaustudip/activities/MensaActivity.java#L94
and here:
https://github.com/ThexXTURBOXx/studip-app-uni-passau/blob/master/app/src/main/res/layout/content_mensa.xml#L11

None of these solutions worked for me...


Web accessibility of tooltipster

I am using tooltipster in one of my project and when checking its accesibily I added to it a role="button" tag to make the screen reader read it (it did not before) but now after it get open once (on hover on the icon) it stays open, Is there any way to make tooltipster web accessible ? that the screen reader would read it and close it when not focused on the icon/div/span that opens it?

Hello, there is a note about accessibility in the use cases section of the documentation, does it work for you?


How can I actually cast my web page?

Hello,
so I registered my TV Cheromecvast and added local webpage as a chanell. How can I cast it to my TV? When I click preview in Devices or Channels, I can only see my wewbpage in the browser only.
Thank you.

From the Devices page (top right), make sure your Chromecast device is visible. Click Register Chromecast, and fill in the form, making sure to select the channel you created as its Channel. Once you save, it should appear on your Chromecast!
If you have any trouble, try restarting Multicast.

What do you see on your TV? Did you follow all the Chromecast Developer Console setup (listed in the readme) to activate your Chromecast as a developer device?

Hello,
I can see only my TVs home screen (app launcher).

Hmm, ok. Let's do some further debugging and see if we can figure out why they're not talking to each other.
Did you open these ports on your host machine?
1900/UDP (DIAL)
5353/UDP (mDNS)
80/TCP/UDP (HTTP)
443/TCP/UDP (HTTPS)

You can add the following lines in app/lib/connection.js to add some basic communication debug logging:
  // line 64
  // monitor receiver status updates to insure hub is open
  d.receiver.on('message', (data, broadcast) => {
+   console.log(Date.now(), data.type, data)
    if (data.type != 'RECEIVER_STATUS')

If you've installed globally, you can run
which multicast

to find your installation directory.
Relaunch Multicast and post your log output after the first 2 minutes or so.

Hello,
I am on Ubuntu (VMWare with bridged connection) and I used sudo ufw allow to open those ports so I guess it should be ok.
Here is a log:
*** WARNING *** The program 'node' uses the Apple Bonjour compatibility layer of Avahi.
*** WARNING *** Please fix your application to use the native API of Avahi!
*** WARNING *** For more information see <http://0pointer.de/blog/projects/avahi-compat.html>
*** WARNING *** The program 'node' called 'DNSServiceRegister()' which is not supported (or only supported partially) in the Apple Bonjour compatibility layer of Avahi.
*** WARNING *** Please fix your application to use the native API of Avahi!
*** WARNING *** For more information see <http://0pointer.de/blog/projects/avahi-compat.html>
MultiCast is live!
Listening at port 3944...
1543254881020 'LAUNCH_ERROR' { reason: 'NOT_FOUND', requestId: 1, type: 'LAUNCH_ERROR' }
connected client with host: <IP_ADDRESS>
disconnected client with host: <IP_ADDRESS>
(node:2434) DeprecationWarning: collection.update is deprecated. Use updateOne, updateMany, or bulkWrite instead.

Thank you.

Are the host and Chromecast on the same network? How is your subnet configured? I notice you're using a non-standard 192.168.100.xxx scheme, it may be that the Chromecast can't resolve that URL.
Open up Chrome from a computer on your network (your host machine, for example) and head to
chrome://inspect

Your developer enabled devices should appear here. Inspect one of them, and then in the console in the window that pops up, attempt to make a fetch request to your host machine (eg - fetch(<IP_ADDRESS>:3944/landing).then(res => res.json()).then(json => console.log(json))). Is that request successful? If not, what sort of error does it throw? You can also check in the Network tab of the inspector to see more details on the request and why it fails.

Just to be sure, in Google Cast SDK App should be IP of my TV and not IP of my Ubuntu machine with MultiCast, correct?

Ah! No, it should be the IP of your Ubuntu machine.

If you get the same error log at first (LAUNCH_ERROR) then you may have to reboot your Chromecast before trying again (so it fetches the updated launch URL from the server).

Sorry, the URL has to include the protocol (http://), try this
fetch('http://<IP_ADDRESS>:3944/landing')
  .then(res => res.json())
  .then(json => console.log(json))

Does your BRAVIA show anything besides the IP address? See the below screenshot:

You should run the fetch code from that inspect link.

No, there is just IP of my TV.
Here is fetch response

And here is Network Preview

I don't understand. I did register my Chromcast device:

Can you paste the contents of your .config file?

{
  "mongoHost": "localhost",
  "mongoPort": 27017,
  "mongoUsername": "mike",
  "mongoPassword": "password",
  "mongoAuthDB": "admin",
  "appId": "xxxxxx"
}

Can you double check and make sure the appId matches the one shown in your Cast Developer Console? In the past, LAUNCH_ERROR has been a result of an appId mismatch.

When you launch Multicast, does your Chromecast TV seem to do anything at all? Does it flicker to a blank white/black screen or show the Multicast green landing page, or no?

@nebev in #34 was also using Sony Bravia TVs with built-in Chromecast and they worked alright, so I don't think that's the problem. I've pushed a handful of minor bugfixes recently without very extensive testing, so it's possible something weird slipped in without my notice. If this is the case, you can try checking out an earlier version and running from there.
git clone https://github.com/superhawk610/multicast.git
cd multicast
# checkout the most recent stable commit
git checkout a2d4b737cc79eeb97e2c85b620b3632f3ff8e400

That's just about all I know to do to debug the current version, but I'm not very happy with it so I am currently working on a complete rewrite using a more modern TypeScript/GraphQL stack. You can track progress on the next branch, but I expect it to be 2 weeks or so before it's usable. Sorry for putting you through so much trouble, we'll definitely find a way to get you up and running! I'll update you on v3.0 progress as it comes.

You don't have to appologize... I am realy thankfull for your help because I learned thing or two. I will try clean installation on some other device (Pi 3B+ probably) and I will be tracking next branch for sure as well.
Thank you again.

I'm closing all v2.x issues that are no longer relevant to the upcoming v3.0 release. The underlying framework is changing so drastically that it's unlikely this bug will be reproducible. Feel free to comment further if this issue persists in v3.0.


chore: Added CodeQL & LGTM static analysis integration

#minor

Coverage remained the same at 54.997% when pulling 23c79a88ba456b4452d3d752a9df27a70356c40a on toanzian/static-analysis into 08bb913b08f89fb200d093679d2559ad0f2bff31 on main.

I've gone ahead and re-enabled the automatic code scanning on LGTM.com (https://lgtm.com/projects/g/microsoft/BotFramework-Composer). I think this satisfies the SDL constraint.

Going to add this to the nightly build instead of the CI build.


7.4.x Possibility to have vim editor in all dockers

Hi there and first of all thanks a lot for this docker
I don't know if I'm doing write...cause it's the first time that I create a pull request.
I install all dockers and when I navigate on my new webserver I directly go in phpmyadmin page.
So I can enter the page with out a password.
I navigate in order to understand how to try to put a password on that webpage
but when I connect to the phpmyadmin docker console (using portainer)
I can't edit the files inside docker.
So the request is... can you make it possible to have vin editor inside the dockers please ?
Thanks again
Denis

Please check out the basics, how github works.
This has nothing todo with what a pull request is for.
The Compose stack shouldn't be used in production. In Production you want to do some things different. Nevertheless you can checkout the env variables of the phpmyadmin todo so checkout: https://hub.docker.com/r/phpmyadmin/phpmyadmin/
Closed since this is not a valid pull request.

Ok thanks a lot :-)


Basic Swap V2

Description
Implementing a new v2 Swap UI into Kwenta: (basic) Synth Exchange.
Related issue
#452
Motivation and Context
Migrate more functions from V1 to V2.
How Has This Been Tested?
Swap different tokens via UI.
Screenshots (if appropriate):

[ ] 1) Navbar: Probably as soon as we merge Exchange, we should rename "Markets" to "Futures" ?
[ ] 2a) Select Synths modal: Can we add a condition to hide the buttons for "Equities"/"Commodities" if there are 0 available?
[ ] 2b) Select Synths modal: There are still a bunch of labels in  elderberry color, we probably want to change them to color: ${(props) => props.theme.colors.common.secondaryGray}; to be in line with the rest of the design

[ ] 3) Just testing it I would have probably have liked it to have the "Select Synths" fields auto-filled on first visit with a default pair, e.g. sUSD-sETH, or maybe at least for "sUSD" to be set as first default maybe. But not sure that is desired, that's a question for product. Similar to Uniswap that shows default ETH and then second pair "Select token".
[ ] 4) Same with color here, elderberry should be secondaryGray. Not sure Price Adjustment applies to L2, so maybe we can hide that info if network != Ethereum ?

[ ] 5) More general question: Is this the default name for the synth by Synthetix? I think this should be Synthetic Crude Oil (WTI)" or something like this, or am I technically buying an Oil PERP here?
[ ] 6) The arrow in the middle of the Synth Exchange modal, I noticed I wanted to click it to swap the two selected synths I had. Is this functionality desired by the design?
For the rest, exchanging synths works like a charm, it looks super clean and just feels totally smooth to use. Great work!
[ ] 7) When I select sUSD, and type an input, e.g. 100 ; and then second step "Select Synth", all the inputs reset to 0. I would want that the 100 sUSD in the first input won't reset, this kind of forces me to go up - down - up in my exchange flow
[ ] 8) PR https://github.com/Kwenta/kwenta/pull/655 removes this check on dev (v1) because dHedge integration does not seem to work using this. We need to check this for v2-dev to make sure dHedge integration works without issues

@platschi I have made the changes based on your feedback (see the above tracklist). Please verify the functionalities. Thanks.

Works absolutely great now, exchange works as expected. A few nitpicks design things:

[ ] Make sure long Synth names have some spacing between lines, see:

[ ] Select Synths: Clicking a button, e.g. in below example "Commodities", works great, but as a user I have no way to know that I need to click one more time on the Commodities button to get back to the "All Synths" overview. I would suggest to implement a similar, subtle effect equal as for the "Long/Short" buttons on Futures Markets, e.g. using transition: all 0.1s ease-in-out 0s and transform: scale(0.98) or so. @asherism what do you think?

[X] Adapt the long synth name in UI

[X] Add a new effect to show if the button is selected

@platschi The following changes have been made.

Updated the selected effect of the categoryButton
Removed "Synth Fees" in tooltip
Showed "Max Fee" on L1 and removed the "EDIT" button
Enabled the input of "Into".

@Tburm  @platschi  The following issues have been fixed.

Update the precision of select currency modal
Increase the width of TxConfirmationModal
Fix the wallet balance display for whales
Remove the unnecessary set method
Reset the input fields after the confirmation of the trades
"Into" input fields only update when a synth selected


Add backend monitoring process

If the backend dies, restart it.
Should tie it to the lifetime of the client, so when the user starts the client we start monitoring, when they close the client we stop monitoring.  I think we can just use clientLifetime for this, but need to test to be sure.
Not 100% sure how to monitor the backend, from what I recall the remote worker has some commands for getting a backend's status, so hopefully it is straightforward.
Essentially this will be an SSH command that runs at an interval, runs the remote worker to check the backend status, and if it is dead it spawns a new one.

Kira to test before we release


VS Code using system language in extensions after installation

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

VS Code Version: 1.71.0
OS Version: Windows 10 Pro 21H2

Steps to Reproduce:

Install VS Code
Install extension (for example cpptools - messages from IntelliSense will be in system default language, not english)
Extension will use system language instead of english which is default, but not in every place

What can be done to workaround this issue:

Install any language pack, switch to that language by using Change Display Language command and return back to english.
Change system default language.

It seems VS Code is not setting it's own display language to english properly, and reports system default language in some places.

I will fold this issue with https://github.com/microsoft/vscode-languageserver-node/issues/1086.
Now that VS Code will have l10n API we need to offer API as well that an extension can find out the locale/UI language.


Retrieving market selection names

Hi all,
how can I retrieve "selection name" from selection_id?
I've seen there is an attribute of RunnerCatalogue called "runner_name". Anyway list_market_catalogue.runners is always an empty list.
Would you be interested in implementing a property of list_market_book.runners for "selection_name"?
Thanks,
Dario

You need to include RUNNER_DESCRIPTION in market_projection in your request

Thanks :)


throw error(500, message) results in "internal error" instead of message when sveltekit() is called elsewhere

Describe the bug
I have a monorepo with several sveltekit subprojects. To recycle my setup I have created a viteconfig that is share among the subprojects. This viteconfig is not inside the same directory as svelte.config.js.
To see the error, execute npx run dev or pnpm exec vite dev. The internal error that you can see in the browser should not be displayed. Instead, the user provided error should be shown.
The bug disappears when sveltekit() is called inside the root directory (as would be the case in the default setup). Looking into the source code, the function sveltekit or load_config depends on the cwd. However, changing the cwd before calling sveltekit() does change the error. Instead of showing the error in the browser using the fallback error page, the browser displays "internal error" and the terminal window show that the server outputs "undefined" when the error is thrown.
This bug means that I have to duplicate my vite config across different subprojects in a monorepo because the sveltekit() function does not follow cwd (when it comes to errors). It seems like other things, apart from the sveltekit error, run.
Reproduction
https://github.com/wvhulle/sveltekit-error-page-vite-plugin
Logs
wvhulle@work ~/D/s/subapp (master) [1]> pnpm run dev

><EMAIL_ADDRESS>dev /home/wvhulle/Documents/sveltekit-error-page-vite-plugin/subapp
> vite dev

  VITE v4.1.1  ready in 1620 ms

  ➜  Local:   http://localhost:3000/
  ➜  Network: use --host to expose
  ➜  press h to show help
undefined
undefined
2:30:08 PM [vite-plugin-svelte] ssr compile done.
package                         	files	  time	   avg
sveltekit-error-page-vite-plugin	    1	29.0ms	29.0ms
@sveltejs/kit                   	    2	 3.9ms	 1.9ms
undefined

### System Info

```Shell
System:
    OS: Linux 5.15 Linux Mint 21.1 (Vera)
    CPU: (8) x64 11th Gen Intel(R) Core(TM) i5-1145G7 @ 2.60GHz
    Memory: 6.90 GB / 15.36 GB
    Container: Yes
    Shell: 3.5.1 - /usr/bin/fish
  Binaries:
    Node: 19.4.0 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 9.2.0 - /usr/local/bin/npm
  Browsers:
    Chromium: 109.0.5414.119
    Firefox: 109.0.1
  npmPackages:
    @sveltejs/adapter-auto: ^2.0.0 => 2.0.0
    @sveltejs/kit: ^1.5.0 => 1.5.6
    svelte: ^3.54.0 => 3.55.1
    vite: ^4.0.0 => 4.1.1

### Severity

annoyance

### Additional Information

I suspect that this bug could be solved with the possibility of an inline config in vite.
https://github.com/sveltejs/kit/issues/5485
However, since changing the cwd does not solve this and the rest keeps working, I suspect this is related more to the implementation of the throw error.

I would like to pass cwd to https://github.com/sveltejs/kit/blob/9259bbad26ae593e82becd74a745214e41f654a9/packages/kit/src/exports/vite/index.js#L113 but it is disabled.

Actually after reading some more, and testing process.cwd, it seems like process.cwd has no effect, so it will have be solved in another way.

It's critical that your app and your config are using the same 'copy' of SvelteKit, otherwise this code, which ensures that error(...) objects are correctly identified as errors that are safe to display to the user...
https://github.com/sveltejs/kit/blob/f0407dce7dd942a643e88ade6a3374041341ba64/packages/kit/src/exports/vite/dev/index.js#L340-L352
...will not work. There are two ways you could do this:

only have @sveltejs/kit installed in the project root, not in subapp etc
use linking: "@sveltejs/kit": "link:../node_modules/@sveltejs/kit" (works with pnpm, other package managers will differ)

Perhaps there's a way we can detect this situation and throw a useful error.

Hi, thanks! I hadn't thought about that. I will test it out. Meanwhile I forked kit and adapted the sveltekit() vite plugin to also take arguments which solves some of these and related problems.


Multiple classes being mangled to "a"

Describe the bug
I have a Base class called BaseModel, and multiple sub-classes like User, Foo, Bar, etc. They all extend BaseModel. The BaseModel class has some logic like:
static _instances = {};

constructor(initialData) {
    if (!BaseModel._instances[this.constructor.name]) {
      BaseModel._instances[this.constructor.name] = [];
    }
    BaseModel._instances[this.constructor.name].push(this);
}

This allows me to have a reference to all instances of the class. The problem is that deploying the app renames all sub-classes to "a" (instead of using "a", "b", etc. for different sub-classes). Hence, my _instances map has multiple model class instances under the same key, causing issues.
So I have:
class User extends BaseModel {}
class Foo extends BaseModel {}
class Bar extends BaseModel {}

But in production code, the this.constructor.name is "a" for all of them (User, Foo, Bar).
Did you try recovering your dependencies?
N/A
Environment
  current version of create-react-app: 5.0.0
  running from /Users/jashsayani/.npm/_npx/c67e74de0542c87c/node_modules/create-react-app

  System:
    OS: macOS 12.1
    CPU: (8) arm64 Apple M1 Pro
  Binaries:
    Node: 16.13.0 - /usr/local/bin/node
    Yarn: Not Found
    npm: 8.1.0 - /usr/local/bin/npm
  Browsers:
    Chrome: 96.0.4664.110
    Edge: Not Found
    Firefox: Not Found
    Safari: 15.2
  npmPackages:
    react: ^17.0.1 => 17.0.1
    react-dom: ^17.0.1 => 17.0.1
    react-scripts: 4.0.1 => 4.0.1
  npmGlobalPackages:
    create-react-app: Not Found

Steps to reproduce

Create class BaseModel and copy constructor from above.
Create sub-classes User, Foo, Bar, etc.
Create instances of User, Foo, Bar, etc. (const f = new Foo();)
Notice that value of this.constructor.name is "a" in constructor (when creating instance of Foo, Bar, etc.) with deploy build code (NOT local server)

Expected behavior
Values of this.constructor.name should be "a", "b", "c", etc. based on different sub-classes
Actual behavior
Creating instances of multiple sub-classes results in this.constructor.name to be "a".

Maybe related: https://github.com/webpack-contrib/uglifyjs-webpack-plugin/issues/394
I see people using eject and overriding webpack config to get around this. I do not want to eject, how can I preserve class names for now? (that seems to be the only solution at this time)
The docs for create-react-app dont mention having a config to override build rules.


Add Enable auto_continue script

Description

Motivation and Context

How has this been tested?

Screenshots (if appropriate)
Types of changes

Changes visible to users:

[ ] Bug fix (prefix: fix - non-breaking change which fixes an issue)
[ ] New feature (prefix: feat - non-breaking change which adds functionality)
[ ] Breaking change (prefix: feat!! or fix!! - fix or feature that would cause existing functionality to not work as expected)

Internal changes:

[ ] Refactor (prefix: refactor - non-breaking change which only improves the design or structure of existing code, and making no changes to its external behaviour)
[ ] Tests (prefix: test - additions and improvements to unit tests and the smoke tests)
[ ] Infrastructure (prefix: chore - examples include GitHub Actions, issue templates)

Checklist

[ ] My change requires a change to the documentation.
[ ] My change works!

Merged your other PR on the Streamlit UI to add this to agent settings which will be more ideal than more scripts in this repo. Thank you!
https://github.com/AGiXT/streamlit/pull/22


Request: Multi-threaded compression

Unless I am completely overlooking something, I see no way to enable multi-threading.
Is there a way to enable this?
If not, is it in the roadmap to support?

There is currently no way to enable multi-threading. This may be added in the near future

In the original ztd library this is achieved using the ZSTD_c_nbWorkers parameter in streaming compression, I can port this.
The code would be like this:
using var input = File.OpenRead("dickens");
using var output = File.OpenWrite("dickens.zst");
using var compressionStream = new CompressionStream(output, level);
compressionStream.SetParameter(ZSTD_cParameter.ZSTD_c_nbWorkers, 4);
input.CopyTo(compressionStream);

@johnsanc314
Is this what you need?

Yes! that's precisely what's needed. I noticed in the code its basically locked down to single thread everywhere, but not sure the implications of allowing multi-threading like you described.
If you can add that it would be a massive performance enhancement.

Added multi-threading support in master

Haha ran into that literally today, i tried to
compressor.SetParameter(ZstdSharp.Unsafe.ZSTD_cParameter.ZSTD_c_nbWorkers, Environment.ProcessorCount);

but got the: "Unsupported parameter"
thanks for adding it!

Released in 0.7.5

Amazing! Thank you for such quick turnaround. It was a long shot even asking, but I'm glad I did. Sounds like several others were wanting this feature as well.


Adds unit test and whitelist for DicomFilter

I accidentally branched this from feat-truncation
The unit test basically implements a simple size check to ensure the filtered version has less content than the non. Later this can be improved, possibly. Not sure how, but surely..
Also all-tests fails with a SIGABRT but they run fine individually.. so I think the file io tests are conflicting with each other in parallel execution.. I don't know what else it could be

So the failures seem to have just been my forgetfulness of the fix-filter branch I had pushed and had not yet merged.

Acquired the dicom images Quinn shared, and the unit tests pass on all the files - all ~6gb of them. Tests pass extremely fast, seemingly parsing and filtering the dicom files with just a couple seconds.


Search button not working after an initial metadata search

Steps to reproduce:

Go to metadata search and search for 'liver`
Select species: Callithrix jacchus and search term: LIAS (from Symbol list)
Click on the Search button
Select species: Homo sapiens and search term: liver (from Metadata list)
Click on the Search button

Expected behaviour:
You get a result for the selected search term filtered by the selected species.
Actual behaviour:
Search is not triggered.

A more general way to generate the bug, is when we switch from gene search to metadata search, the search button is not working.

I did a deeper research and some hand-on tests, and found out that, in single-cell, we have gene search result page and cell type wheel result page, where both have gene-search-form component respectively.
However, the metadata search GetMapping controller is directed by search/metadata/liver for example, wheras, gene search GetMapping controller is directed by search?q=liver.

The error is generated in this way:

Once we click the metadata search example from the homepage or search a metadata value from the homepage search box, it will direct us to the cell-type-wheel-heatmap result page.
If we search a non-metadata from the cell-type-wheel-heatmap result page, it will direct us to search?, .i.e gene search result page, another component.
However, in gene search result page, the gene search form didn't deal with the metadata search properly, if you make a metadata search, it will direct us, i.e. the URL is search?metadata=liver rather than search/metadata/liver. Therefore, we can't get any cell-type-wheel data from a gene search result page search form.

There are two solutions I proposed like below:

Component-level solution:

[ ] Combine the gene search result page and cell type wheel page into one component, which sharing a single use of gene-search-form component.
[ ] Unify the controller format for metadata and gene search, either take the term as a PathVariable or RequestParam

Router-level solution:

[ ] In gene search result page component, add a router for metadata search, redirect to search/metadata/[term]

@lingyun1010 I discussed this with Silvie and she agreed that for this release the quicker solution (2nd) would be preferred and we should create a ticket for the 1st solution and add it to our backlog for technical dept.
Thanks


Allow exports-type on artifacts

Codecov Report

Merging #89 (174901c) into master (d3b3fba) will decrease coverage by 0.05%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master      #89      +/-   ##
==========================================
- Coverage   87.89%   87.83%   -0.06%
==========================================
  Files          39       39
  Lines        1314     1324      +10
  Branches       73       74       +1
==========================================
+ Hits         1155     1163       +8
- Misses        132      134       +2
  Partials       27       27

Impacted Files
Coverage Δ

...nan/bazel/mvn/api/model/ExportsGenerationType.java
45.45% <25.00%> (+45.45%)
:arrow_up:

...vendanan/bazel/mvn/api/model/ResolutionOutput.java
100.00% <100.00%> (ø)

...vendanan/bazel/mvn/api/serialization/Adapters.java
98.91% <100.00%> (+0.03%)
:arrow_up:

Continue to review full report at Codecov.

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


Makefile not working on Mac

I am working on #4146 and when I run make generate to build my CRDs and all other auto-generated files, I get the following output:
go install<EMAIL_ADDRESS>Installing k8s.io/code-generator tools with version v0.25.6
#go install<EMAIL_ADDRESS>go install github.com/astefanutti/code-generator/cmd/applyconfiguration-gen@pr-144-install
go install<EMAIL_ADDRESS>go install<EMAIL_ADDRESS>go install<EMAIL_ADDRESS>./script/gen_client.sh
Generating Go client code...
./script/gen_crd.sh
sed: 1: "/creationTimestamp:/a\  ...": extra characters after \ at the end of a command
make: *** [generate] Error 1

When I look at the auto-generated files, I notice this:

It doesn't include the comments regarding the license
It adds the unnecessary ---
creationTimestamp colored-out, not sure what that means
labels not included

I change the following code in gen_crd.sh
  sed -n '/^---/,/^status/p;/^status/q' "${source}.orig" \
    | sed '1d;$d' \
    | sed '/creationTimestamp:/a\  labels:\n    app: camel-k' >> "$source"

to the following one-liner:
sed -n '/^---/,/^status/p;/^status/q' "${source}.orig" '1d;$d' '/creationTimestamp:/a\ labels:\n app: camel-k' >> "$source"

Which helps with the issue I face when running make generate.
Generating CRD API documentation...
I0328 07:35:37.555490   33758 main.go:136] parsing go packages in directory github.com/apache/camel-k/v2/pkg/apis/camel/v1
I0328 07:35:39.506643   33758 main.go:240] using package=github.com/apache/camel-k/v2/pkg/apis/camel/v1
I0328 07:35:39.506666   33758 main.go:240] using package=github.com/apache/camel-k/v2/pkg/apis/camel/v1/trait
W0328 07:35:39.525619   33758 main.go:469] not found external link source for type encoding/xml.Name
I0328 07:35:39.529658   33758 main.go:176] written to ./script/gen_crd/../../docs/modules/ROOT/partials/apis/camel-k-crds.adoc
I0328 07:35:40.300064   33850 main.go:136] parsing go packages in directory github.com/apache/camel-k/v2/pkg/apis/camel/v1alpha1
I0328 07:35:42.243031   33850 main.go:240] using package=github.com/apache/camel-k/v2/pkg/apis/camel/v1alpha1
W0328 07:35:42.255036   33850 main.go:469] not found external link source for type encoding/json.Number
W0328 07:35:42.255111   33850 main.go:469] not found external link source for type encoding/json.Number
W0328 07:35:42.255200   33850 main.go:469] not found external link source for type encoding/json.Number
W0328 07:35:42.255542   33850 main.go:469] not found external link source for type github.com/apache/camel-k/v2/pkg/apis/camel/v1.PodCondition
W0328 07:35:42.255616   33850 main.go:469] not found external link source for type github.com/apache/camel-k/v2/pkg/apis/camel/v1.IntegrationSpec
W0328 07:35:42.256015   33850 main.go:469] not found external link source for type github.com/apache/camel-k/v2/pkg/apis/camel/v1.SourceSpec
I0328 07:35:42.256471   33850 main.go:176] written to ./script/gen_crd/../../docs/modules/ROOT/partials/apis/kamelets-crds.adoc
Generating CRD API documentation... Done.
Generating API documentation... done!
Generating traits documentation...
Generating traits documentation... done!
cd pkg/apis/camel && /Users/mert/go/bin/controller-gen paths="./..." object
cd addons/keda/duck && /Users/mert/go/bin/controller-gen paths="./..." object
cd addons/strimzi/duck && /Users/mert/go/bin/controller-gen paths="./..." object
./script/gen_client_strimzi.sh
Generating Go client code for Strimzi addon...

So it seems to work. No errors at least. But when I look at the auto-generated files, they are still not 100% correctly generated.

Still, the labels is missing.
Any idea why? Any Mac users who are able to run the Makefile scripts with no issues? If so, did you do any Mac-specific configurations?
Many thanks in advance!

You need to ensure that gnu-sed is used, please read the note in the requirements section https://camel.apache.org/camel-k/next/contributing/developers.html#requirements

Alright, I missed that specific note for MacOS users in the documentation. Followed those steps and it solved my issue. Thanks @essobedo!


window explorer (and terminal) hangs when I edit the address bar in a file picker

Windows Terminal version
1.19.10573.0
Windows build number
10.0.19045
Other Software
Windows explore

Steps to reproduce
In the icon option for editing a single profile, after using the browse option and opening window explorer, it is not possible to enter the path to the file directly in the address bar, it is not possible to enter any characters, which causes the entire Windows terminal to go down, and the only way to close the program is to close it through task management.
please look the gif animation
Expected Behavior

Actual Behavior
software is dead

Hmm. We once had a similar report in #8957, but that was when users were running the Terminal elevated, and was also fixed in like, 1.7.
There was also #9984, which sounds almost exactly the same, but OP disappeared on that one.
I also can't repro this here on Windows 11.
Filing /feedback with the Feedback hub (but with the area set to like, "Files and Online Storage > File Explorer") may work to get a trace:

Here's a link to the feedback recorded in the feedback center
Does this have anything to do with the network driver clouddriver2 that I installed

Huh. Interestingly enough, it looks like you're the only one who's hit this?

Apparently I can't promote the feedback hub feedback to an MSFT: work item. That's weird, but I think we're changing some backend bits.
I can still get to feedback ID 9a191c39-d40a-4559-82ed-d23e3cfec3a1
It's got three .dmps for explorer.exe,
and a link to a bucket:<PHONE_NUMBER>875420617 (BucketDetails?BucketIdentifier=1568601951875420617).

There's only exactly one hit for this specific bucket in the last 90 days.
It's a moapphang in microsoft.windowsterminal_1.19.10821.0_x64__8wekyb3d8bbwe, and we have other moapphang's hanging around too, but I don't think they're related
There's nothing else interesting in the bucket...

Dustin repro'd this instantaneously on Windows 10, and Leonard thinks... I think this code is the problem:
https://github.com/microsoft/terminal/blob/49e4eea60f737b46b8aeda505f4693df8a9d44a6/src/cascadia/WinRTUtils/inc/Utils.h#L15-L55
https://github.com/microsoft/terminal/blob/49e4eea60f737b46b8aeda505f4693df8a9d44a6/src/cascadia/TerminalSettingsEditor/Profiles_Base.cpp#L109-L125
Those sorta attempt to do the work async, but actually don't make it async at all

Really? Did you locate the bug so quickly, and how long will it take to fix it, please?

Really? Did you locate the bug so quickly, and how long will it take to fix it, please?

I have no idea if that's the actual source of the bug or not. We had coincidentally just been chatting about how this particular code didn't work as expected. And it sure seems like this is the kind of thing that might cause this bug.
No ideas how long it would take to fix. It might not even be the correct source of the bug 🤷


Figure out how to handles chunks

https://github.com/PrismarineJS/prismarine-world/blob/master/src/world.js

maybe https://github.com/PrismarineJS/prismarine-provider-anvil/blob/master/src/module.js

implement prismarine-world https://github.com/PrismarineJS/prismarine-world/issues/2
then use it.

Done, the rest of the work (generation and stuff) should be tracked in #10


Backport "HBASE-24658 Update PolicyBasedChaosMonkey to handle uncaught exceptions" to branch-1

Running ServerKillingChaosMonkey via RESTApiClusterManager for any
duration of time slowly leaks region servers. I see failures on the
RESTApi side go unreported on the ChaosMonkey side. It seems like
RuntimeExceptions are being thrown and lost.
PolicyBasedChaosMonkey uses a primitive means of thread management
anyway. Update to use a thread pool, thread groups, and an
uncaughtExceptionHandler.
Signed-off-by: Bharath Vissapragada<EMAIL_ADDRESS>Signed-off-by: Viraj Jasani<EMAIL_ADDRESS>

Posted a patch for buildbot, as I'm out of practice on branch-1. FYI @bharathv , @virajjasani in case you want to see the backport. Shouldn't be anything surprising here.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
11m 53s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 1 new or modified test files.

_ branch-1 Compile Tests _

+1 :green_heart:
mvninstall
9m 53s
branch-1 passed

+1 :green_heart:
compile
0m 21s
branch-1 passed with JDK v1.8.0_252

+1 :green_heart:
compile
0m 27s
branch-1 passed with JDK v1.7.0_272

+1 :green_heart:
checkstyle
0m 41s
branch-1 passed

+1 :green_heart:
shadedjars
3m 20s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 27s
branch-1 passed with JDK v1.8.0_252

+1 :green_heart:
javadoc
0m 14s
branch-1 passed with JDK v1.7.0_272

+0 :ok:
spotbugs
4m  7s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
0m  0s
branch-1 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m  6s
the patch passed

+1 :green_heart:
compile
0m 21s
the patch passed with JDK v1.8.0_252

+1 :green_heart:
javac
0m 21s
the patch passed

+1 :green_heart:
compile
0m 27s
the patch passed with JDK v1.7.0_272

+1 :green_heart:
javac
0m 27s
the patch passed

+1 :green_heart:
checkstyle
0m 31s
hbase-it: The patch generated 0 new + 1 unchanged - 1 fixed = 1 total (was 2)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
3m  5s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
5m  2s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2.

+1 :green_heart:
javadoc
0m 12s
the patch passed with JDK v1.8.0_252

+1 :green_heart:
javadoc
0m 15s
the patch passed with JDK v1.7.0_272

+1 :green_heart:
findbugs
0m  0s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m 10s
hbase-it in the patch passed.

+1 :green_heart:
asflicense
0m 20s
The patch does not generate ASF License warnings.

44m 21s

Subsystem
Report/Notes

Docker
Client=19.03.12 Server=19.03.12 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2108/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2108

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux 4e20d1980392 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/Base-PreCommit-GitHub-PR_PR-2108/out/precommit/personality/provided.sh

git revision
branch-1 / d697c40

Default Java
1.7.0_272

Multi-JDK versions
/usr/lib/jvm/zulu-8-amd64:1.8.0_252 /usr/lib/jvm/zulu-7-amd64:1.7.0_272

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2108/1/testReport/

Max. process+thread count
481 (vs. ulimit of 10000)

modules
C: hbase-it U: hbase-it

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2108/1/console

versions
git=1.9.1 maven=3.0.5

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 13s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 1 new or modified test files.

_ branch-1 Compile Tests _

+1 :green_heart:
mvninstall
9m 55s
branch-1 passed

+1 :green_heart:
compile
0m 21s
branch-1 passed with JDK v1.8.0_252

+1 :green_heart:
compile
0m 28s
branch-1 passed with JDK v1.7.0_272

+1 :green_heart:
checkstyle
0m 41s
branch-1 passed

+1 :green_heart:
shadedjars
3m 21s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 28s
branch-1 passed with JDK v1.8.0_252

+1 :green_heart:
javadoc
0m 16s
branch-1 passed with JDK v1.7.0_272

+0 :ok:
spotbugs
4m 10s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
0m  0s
branch-1 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m  5s
the patch passed

+1 :green_heart:
compile
0m 21s
the patch passed with JDK v1.8.0_252

+1 :green_heart:
javac
0m 21s
the patch passed

+1 :green_heart:
compile
0m 27s
the patch passed with JDK v1.7.0_272

+1 :green_heart:
javac
0m 27s
the patch passed

+1 :green_heart:
checkstyle
0m 31s
hbase-it: The patch generated 0 new + 1 unchanged - 1 fixed = 1 total (was 2)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
3m  8s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
4m 58s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2.

+1 :green_heart:
javadoc
0m 13s
the patch passed with JDK v1.8.0_252

+1 :green_heart:
javadoc
0m 17s
the patch passed with JDK v1.7.0_272

+1 :green_heart:
findbugs
0m  0s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m 23s
hbase-it in the patch passed.

+1 :green_heart:
asflicense
0m 22s
The patch does not generate ASF License warnings.

34m  5s

Subsystem
Report/Notes

Docker
Client=19.03.12 Server=19.03.12 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2108/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2108

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux 0dba2c076a2d 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/Base-PreCommit-GitHub-PR_PR-2108/out/precommit/personality/provided.sh

git revision
branch-1 / d697c40

Default Java
1.7.0_272

Multi-JDK versions
/usr/lib/jvm/zulu-8-amd64:1.8.0_252 /usr/lib/jvm/zulu-7-amd64:1.7.0_272

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2108/2/testReport/

Max. process+thread count
479 (vs. ulimit of 10000)

modules
C: hbase-it U: hbase-it

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2108/2/console

versions
git=1.9.1 maven=3.0.5

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.


Patch for /LICENSE

anand.almora.04 on 16 Jun 2014 at 9:29:
asasas

Successfully auto-migrated from code.google.com:
https://code.google.com/p/google-api-go-client/issues/detail?id=60

Spam.  Closing.


Added ability to call analytics report manually

Motivation and context

How has this been tested?

Checklist

[x] I submit my changes into the develop branch
[x] I have created a changelog fragment
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (see GitHub docs)
[ ] I have increased versions of npm packages if it is necessary
(cvat-canvas,
cvat-core,
cvat-data and
cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.

Summary by CodeRabbit

New Features

Added new filter types and a calculate function for analytics, quality reports, conflicts, and settings in the API.
Introduced a new method to calculate performance analytics.
Enhanced analytics page UI with new functions and updated styling for better user interaction.
Added a button to request analytics report calculation on the analytics page.

Refactor

Improved naming conventions and refactored import statements for better organization and readability in server proxy and permissions modules.

Style

Updated styles for the analytics overview and refresh button to improve visual layout.

Documentation

Updated in-code documentation to reflect changes in API functionalities and UI components.

Bug Fixes

Fixed issues related to the initialization of analytics reports to ensure proper handling of missing IDs.

@coderabbitai review

@coderabbitai review

While testing the patch the clickhouse was not up on my machine and the error occured. It seems full stack trace is shown right in the notification. I cant even scroll it 😅 No problem with this exact error but are all the possible errors are displayed this way?

Stacktraces should not be exposed to client
Now we have several expressions, like raise ValidationError(ex)
That is not related to this PR, but I will submit one more to fix it, also partially I aldready fixed it in another PR #7804, but perhaps  this case is not handled there likewise

While testing the patch the clickhouse was not up on my machine and the error occured. It seems full stack trace is shown right in the notification. I cant even scroll it 😅 No problem with this exact error but are all the possible errors are displayed this way?

Checked one more time.
It is a problem with code in endpoint POST /analytics/report?rq_id=<...>
Now the endpoint does not return enough data and will be deprecated/removed after the patch with requests is finished.


V1.1

New Widget Added

New Widget Added

Apologies. .. by mistake opened an pr on the base branch


footer icon added issues: #309

Active Member url changed "/activeMembers" to "/active-members". Deleted duplicated activeMembers page.
footer icon added - github, twitter, discord

Great job Sakib!! Although it would be an even better PR of you made the icons links to the respective platforms.

@devvsakib

@tobySolutions  omg, I forgot to add links due to re fork and solving problems 🤣🤣🤣.. thank youuuu

added link

Great job @devvsakib!!


Unable to load AWS credentials from any provider in the chain

I am trying to run the RedshiftBasic sample and I have included aws.accessKeyId and aws.secretKey properties at the end of the RedshiftBasicSample.properties file per the instructions. When I run 'ant run', the following trace comes out:
run:
    [javac] /Users/andrewguy/www/amazon-kinesis-connectors/src/main/samples/redshiftbasic/build.xml:48: warning: 'includeantruntime' was not set, defaulting to build.sysclasspath=last; set to false for repeatable builds
    [javac] Compiling 28 source files to /Users/andrewguy/www/amazon-kinesis-connectors/src/main/build
    [javac] warning: Supported source version 'RELEASE_6' from annotation processor 'com.amazonaws.eclipse.simpleworkflow.asynchrony.annotationprocessor.AsynchronyDeciderAnnotationProcessor' less than -source '1.8'
    [javac] 1 warning
     [java] Exception in thread "main" com.amazonaws.AmazonClientException: Unable to load AWS credentials from any provider in the chain
     [java]     at com.amazonaws.auth.AWSCredentialsProviderChain.getCredentials(AWSCredentialsProviderChain.java:117)
     [java]     at com.amazonaws.services.kinesis.AmazonKinesisClient.invoke(AmazonKinesisClient.java:2486)
     [java]     at com.amazonaws.services.kinesis.AmazonKinesisClient.describeStream(AmazonKinesisClient.java:861)
     [java]     at samples.utils.KinesisUtils.streamExists(Unknown Source)
     [java]     at samples.utils.KinesisUtils.createAndWaitForStreamToBecomeAvailable(Unknown Source)
     [java]     at samples.utils.KinesisUtils.createInputStream(Unknown Source)
     [java]     at samples.KinesisConnectorExecutor.setupAWSResources(Unknown Source)
     [java]     at samples.KinesisConnectorExecutor.<init>(Unknown Source)
     [java]     at samples.redshiftbasic.RedshiftBasicExecutor.<init>(Unknown Source)
     [java]     at samples.redshiftbasic.RedshiftBasicExecutor.main(Unknown Source)
     [java] Java Result: 1

BUILD SUCCESSFUL
Total time: 3 seconds

What am I missing?
Thanks.

Update:
this approach worked:
$ export AWS_ACCESS_KEY_ID=...
$ export AWS_SECRET_ACCESS_KEY=...

However, when I have a production ready system, this approach will not work for me. Any reason, why the properties file is not taking my access ids?

I solved this adding those variables as root user.
$ sudo su
$ mkdir /root/.aws
$ nano credentials

# After you create the file, exit as root user
$ exit

Status :FAILED
Reason :
Log File : s3://aws-logs-388546034523-ap-southeast-2/elasticmapreduce/j-3MMYU5AZJ1UNE/steps/s-2B4ONS94M7B00/stderr.gz
Details : Exception in thread "main" com.amazon.ws.emr.hadoop.fs.shaded.com.amazonaws.SdkClientException: Unable to load AWS credentials from any provider in the chain
JAR location : command-runner.jar
Main class : None
Arguments : spark-submit --class com.example.project.SimpleApp --deploy-mode cluster --master yarn s3://XXXXX/spark-log-job_2.10-1.0.jar
Action on failure: Continue
Hi ,
I am facing this issue when i submit  spark  scala job .
looking for some help on this .
Thanks in Advance .

I am still facing this issue, i am running docker container and trying to connect to DynamoDB, when i curl on one of the end point i get "message":"com.amazonaws.SdkClientException: Unable to load AWS credentials from any provider in the chain
i am logging in through saml2aws and credentials are already present in .aws folder. Please advise

I'm having a similar issue running kcl-bootstrap (https://github.com/awslabs/amazon-kinesis-client-nodejs). It seems like it's unable to read credentials in ~/.aws/credentials and fails with SdkClientException: Unable to load AWS credentials from any provider in the chain, although my credentials setup is working elsewhere. Same as mentioned above, if I directly set AWS_ACCESS_KEY_ID and AWS_SECRET_ACCESS_KEY, it does work. It's like the environment isn't being passed through somewhere.


support for react 15

Closes #5
@karlbright Can you have a Look ? Thx!

@d7h Sounds good. If you could fix up the breaking tests then I'll happily merge. :+1:

@karlbright ok, in this context i've updated all the dependencies

Good job :+1:

@d7h Just published 0.6.0 with this change

@karlbright Thx!


Integration does not work for EV6 (Kia Connect) in Sweden (EU)

Please check Services, Known Bug / Issues and Troubleshooting over here first: https://github.com/fuatakgun/kia_uvo/blob/master/README.md
Region and Brand of car
Kia EV6
Describe the bug
A clear and concise description of what the bug is.
Cannot log in,
Debug logs if an error occurred
Outlined here: https://github.com/fuatakgun/kia_uvo/blob/master/README.m
To Reproduce
Steps to reproduce the behavior:
Error message
"Login failed into Kia Uvo / Hyundai Bluelink Servers. Please use official Kia Uvo app to logout and log back in and try again!"
If I follow instructions, no change. I have reinstalled the integration from HACS.
Expected behavior
A clear and concise description of what you expected to happen.
When logging in from GUI an error message pop up.
Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
Add any other context about the problem here.
Pulled from the log
Logger: custom_components.kia_uvo.config_flow
Source: custom_components/kia_uvo/config_flow.py:170
Integration: Kia Uvo / Hyundai Bluelink (documentation, issues)
First occurred: 17:20:01 (1 occurrences)
Last logged: 17:20:01
kia_uvo Exception in kia_uvo login : HTTPSConnectionPool(host='prd.eu-ccapi.kia.com', port=8080): Max retries exceeded with url: /api/v1/spa/notifications/register (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7f5547a10760>: Failed to establish a new connection: [Errno 110] Operation timed out')) - traceback: Traceback (most recent call last): File "/usr/local/lib/python3.9/site-packages/urllib3/connection.py", line 174, in _new_conn conn = connection.create_connection( File "/usr/local/lib/python3.9/site-packages/urllib3/util/connection.py", line 95, in create_connection raise err File "/usr/local/lib/python3.9/site-packages/urllib3/util/connection.py", line 85, in create_connection sock.connect(sa) TimeoutError: [Errno 110] Operation timed out During handling of the above exception, another exception occurred: Traceback (most recent call last): File "/usr/local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 703, in urlopen httplib_response = self._make_request( File "/usr/local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 386, in _make_request self._validate_conn(conn) File "/usr/local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 1040, in _validate_conn conn.connect() File "/usr/local/lib/python3.9/site-packages/urllib3/connection.py", line 358, in connect conn = self._new_conn() File "/usr/local/lib/python3.9/site-packages/urllib3/connection.py", line 186, in _new_conn raise NewConnectionError( urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPSConnection object at 0x7f5547a10760>: Failed to establish a new connection: [Errno 110] Operation timed out During handling of the above exception, another exception occurred: Traceback (most recent call last): File "/usr/local/lib/python3.9/site-packages/requests/adapters.py", line 439, in send resp = conn.urlopen( File "/usr/local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 785, in urlopen retries = retries.increment( File "/usr/local/lib/python3.9/site-packages/urllib3/util/retry.py", line 592, in increment raise MaxRetryError(_pool, url, error or ResponseError(cause)) urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='prd.eu-ccapi.kia.com', port=8080): Max retries exceeded with url: /api/v1/spa/notifications/register (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7f5547a10760>: Failed to establish a new connection: [Errno 110] Operation timed out')) During handling of the above exception, another exception occurred: Traceback (most recent call last): File "/config/custom_components/kia_uvo/config_flow.py", line 155, in async_step_user self.token = await self.hass.async_add_executor_job( File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 52, in run result = self.fn(*self.args, **self.kwargs) File "/config/custom_components/kia_uvo/KiaUvoApiEU.py", line 101, in login self.device_id, self.stamp = self.get_device_id() File "/config/custom_components/kia_uvo/KiaUvoApiEU.py", line 167, in get_device_id response = requests.post(url, headers=headers, json=payload) File "/usr/local/lib/python3.9/site-packages/requests/api.py", line 117, in post return request('post', url, data=data, json=json, **kwargs) File "/usr/local/lib/python3.9/site-packages/requests/api.py", line 61, in request return session.request(method=method, url=url, **kwargs) File "/usr/local/lib/python3.9/site-packages/requests/sessions.py", line 542, in request resp = self.send(prep, **send_kwargs) File "/usr/local/lib/python3.9/site-packages/requests/sessions.py", line 655, in send r = adapter.send(request, **kwargs) File "/usr/local/lib/python3.9/site-packages/requests/adapters.py", line 516, in send raise ConnectionError(e, request=request) requests.exceptions.ConnectionError: HTTPSConnectionPool(host='prd.eu-ccapi.kia.com', port=8080): Max retries exceeded with url: /api/v1/spa/notifications/register (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7f5547a10760>: Failed to establish a new connection: [Errno 110] Operation timed out'))

Please double check if you have an internal firewall blocking this call, in essence, your home assistant instance is failing to reach kia/hyundai servers.

Hmm, cant be that. I can ping the server from Terminal i HomeAssistant,

Ping confirms routing not connectivity.  Please try telnet. Also try the mobile app while on wifi or the same network.

On Jan 25, 2022, at 10:46 AM, a94marbo @.***> wrote:

Hmm, cant be that. I can ping the server from Terminal i HomeAssistant,
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.

Hmm.. ok from WIFI it does not work from the app. Had som old port forwarding rules for port 8080, but still does not work after that I have removed them. But something is fishy... thanks... I will have to keep hunting.

Sorry for reopening this issue.
I realize that my router is causing this, but I cannot get it to work.
I have port forwarded port 8080 to the home assistant computer, but I still get this problem.
I have experimented a bit with the phone app, On mobile network it works, on WIFI it does not work, on WIFI with VPN on it works, Is only communication on port 8080?

We don't have control of kia servers and their port requirements, we need to hit their 8080 port.

I understand that. So it should be enough to port forward 8080 to home assistant?

No this is the wrong direction - home assistant (and the app) do an outbound connection to Kia/hyundai.  You should not need any inbound port forwarding configured.   What it sounds like is happening is port 8080 is being blocked outbound.  This is separate from port forwarding.  I am concerned for your networks security as port forwarding has significant risk if not used correctly.
Read up on the concept of a NAT as I assume you are using that.   A default consumer device will allow all outbound communication and this will work for this integration.  Port forwarding in a consumer circumstance is only ever used when you are hosting something that you want everyone on the internet to access.  Including strangers.

If you have more questions feel free to comment.  For now I will close this.


So.. where is the viz? :D

It’s a work in progress, lol. Patience. 🧘♂

❤ 🐻

@abitrolly so to keep you in the loop - I have scripts that convert the caliper output to csv formatted (or whatever we like really) and the strategy I want to take now is using grafana. I have it loaded as a data source, but I don't understand the use cases ;/ graphs that someone would want to see, and more specifically how that maps to the caliper output. Do you have a sense?

It it is this Caliper - http://software.llnl.gov/Caliper/ - which is about performance analysis, then I'd expect some performance graphs like https://observablehq.com/@cimi/jfr-flamegraph?collection=@cimi/java-mission-control from it.

Perfecto! I'll do that one first.

okay got a basic flamegraph, although the structure is wrong. I think we should use the work by Pascal that has gone into spot for these vis, so my gameplan now is to first do this https://github.com/LLNL/spotbe/issues/18 and then start to refactor what is currently spot into a more streamlined thing that just loads from the database and then makes the graphs.

Hmm... Performance visualization libraries without visualizations don't looks good. Obviously.  I don’t appreciate it.

The username should check out if I do, but I am actually speaking of https://github.com/LLNL/spotbe and related repos that lack examples.

Going to close this issue, thanks for your interest @abitrolly.

Ping me once you have some screens. There is no way I could use that myself, but I am curious to see what LLNL is cooking.


fix defer in loop, maybe resource leak

fix defer in loop, maybe resource leak

Good catch, certainly looks like a resource leak to me as well. Also want to point out the change in benchmark behavior. Before the benchmark would time the startup of N tasks synchronously; however, now the measurement will include cleanup. Not sure the intent but maybe someone can chime in?

I'm not sure we want to merge this. This breaks the benchmark.


EODS search parameters is returning empty queries

The search parameters seem to no longer be working. I opened up by notebook (which worked last week) and now it is only returning empty responses. I have tried several date variations etc. to make sure it wasn't an issue with the dates but I am getting the same issue.
I checked out the example notebooks to see if the issue persisted there also, and it did: eods-example1-simple-EODS-query

I left everything standard apart from editing the .env file with my token ID
any help would be appreciated
Thanks
Ty

Hi @tyhayward,
I'm also having the same issue on my end. It appears to be connected to the use of the geometry search functionality.
I'll get back to you when I know more.
Thanks,
Henry

Hi @tyhayward,
Apologies for the delay in getting back to you. Thank you for bringing this to our attention. After further investigation it appears that the geometry search functionality is no longer working on the server side, and this outage has nothing to do with the code base.
An internal ticket has been raised and it is being currently worked.
In the meantime, I would recommend removing the geometry search parameter from your code

@tyhayward If you add this to your code after you retrieve list_of_layers and df from query_catalog, it should filter out all the layers which do not intersect with the England polygon you shared in a previous issue.
england_list = ['T29UPR','T29UQR','T29UQS','T30UUA','T30UUB','T30UVA','T30UVB','T30UVC','T30UVD','T30UVE','T30UVF','T30UVG','T30UWA','T30UWB','T30UWC','T30UWD','T30UWE','T30UWF','T30UWG','T30UXA','T30UXB','T30UXC','T30UXD','T30UXE','T30UXF','T30UXG','T30UYB','T30UYC','T30UYD','T30UYE','T31UCS','T31UCT','T31UCU','T31UCV','T31UDS','T31UDT','T31UDU','T31UDV']
df = df[df['granule-stub'].isin(england_list)]
filtered_list_of_layers = [i for i in list_of_layers if i.split('_')[3][:6] in england_list]
list_of_layers = filtered_list_of_layers

@henrywild thanks ill give it a go


ngc doesn't allow components in source that aren't being used

If you have a @Component defined somewhere, but the component isn't being declared in NgModule, ngc complains with
Error: cannot determine the module for component MyComponent
This means you can't really work on partial components without immediately including them in NgModule.

@mlynch I am working on updating the framework to the latest NGC and Angular which may include a fix for this here.

Pretty sure we have a fix for this that will go live with the app-scripts release. It will involve adding an include statement to tsconfig.
I'll verify and update at some point.
Thanks,
Dan

@mlynch,
This is fixed as long as the module isn't imported anywhere. I think that's a reasonable behavior.
Thanks,
Dan


Service Error 401002 CA certificate not found error

Hello,
I executed following steps

Create CA certificate using create cert chain.
Uploaded CA certificate to certificates section under DPS.
Generated verification code.
Generated verification certificate using verification code as common name using createverificationcert
uploaded verification certificate and verified successfully.
6)created group enrollment with Root certificate.
7)generated device specif pfx using createdevicecert
8)tried to provision device using device specific certificate it throws Service Error $01002 Certificate not found (.Net Simulated device)

Hi Shreyasnie,
You need to check that you are sending also certificate shain to DPS.
The constructor SecurityProviderX509Certificate is taking two parameters, device certificate and chain, you need to provide both.
I had the same problem and figured out that this was missing.
Best Regards,
Dusan

@Shreyasnie: If @dusanruzicic's suggestion did not help you, then please write again.


ARROW-2445: [Rust] Add documentation and make some fields private

No functional changes.

@pitrou or @xhochy Could you please merge these changes?


"Parent Stream too short" when message size exceeds 100 bytes

Hey there,
I am decoding my messages by calling pb_istream_from_buffer and then pb_decode after each char is received.   This works fine for small messages, but when the message size exceeds 100 bytes, pb_decode throws the error "parent stream too short".
I haven't been able to understand the documentation or source code to figure out exactly what is going on.  Something to do with substreams that is over my head.
Do you have any advice?
This is my message receiving code:
void receive_message(pb_byte_t stream_buffer[],const pb_field_t message_fields[], msgType *message_ptr) {
  size_t bytes_read = 0;
  while (1) {
    if (pc_.readable()) {

      // Create a stream that reads from the buffer.
      char c = pc_.getc();
      stream_buffer[bytes_read] = c;
      bytes_read++;

      // Try decoding every time
      pb_istream_t stream = pb_istream_from_buffer(
          static_cast<pb_byte_t *>(stream_buffer), bytes_read);

      // Now we are ready to decode the message.
      bool status =
          pb_decode(&stream, message_fields, message_ptr);

      // Check for errors...
      if (status) {
        // Decode succeeded. Message is now ready for use.
        return;
      }

      // Take another character and try again.
    }
  }
}

What is the actual message data you are trying to decode, e.g. as a hex dump?
If you are on Windows, make also sure that the pc_ stream is set to binary mode, to avoid Windows from altering 0x0A byte values.
But in reality, a protobuf message may decode successfully even before it has been fully received, so I would recommend more explicit way to pass the message length. Such as pb_encode_delimited().

Thanks for the timely reply.
I am running ubuntu, and here is the hex dump I get from print(msg.SerializeToString()):
b'\n\x1a\x08\x00\x10\x00\x18\x00 \x00(\x000\x008\x8b\x1a@\x9f\x10H\x00P\x00X\x00\x00\x12\x18\x08\xeb\x03\x10\xff\x1f\x18\x99\x13 \x8e\x11(\x000\x008\x00@\x00H\x00P\x01\x1a\x1a\x08\x00\x10\x00\x18\x00 \x00(\x000\x008\x8b\x1a@\x9f\x10H\x00P\x00X\x00\x00"\x18\x08\xeb\x03\x10\xff\x1f\x18\x99\x13 \x8e\x11(\x000\x008\x00@\x00H\x00P\x01'
Ok I will look into pb_encode_delimited().  In the case a message is decoded before fully received, I suppose I should flush the serial buffer to avoid getting the leftover bytes on the next message.

That message data appears corrupted, for example protoc --decode_raw does not accept it.

I tested the data integrity by passing back each char received over the
same pc_ serial port and inspecting on the python side.  It all checks
out.
Can you elaborate what you mean by correctly setting to binary mode?
On Thu, Nov 10, 2022, 8:39 AM Petteri Aimonen @.***>
wrote:

Support for delimiting varies a between between libraries, e.g.
https://github.com/protocolbuffers/protobuf/blob/main/src/google/protobuf/util/delimited_message_util.h
has it for C++. I think the Python protobuf library still doesn't have
support built-in, though e.g. https://pypi.org/project/delimited-protobuf/
provides it. The general idea is mentioned in docs here:
https://developers.google.com/protocol-buffers/docs/techniques#streaming
Actually the data seems to be accepted by protoc --decode_raw fine now.
Before your edit the Github formatting was just eating out the ` so I got a
corrupted 106 byte message instead of the correct 108 bytes.
You may want to verify the data you are actually passing to pb_decode()
also, to make sure there is no further corruption in between. Especially
serial ports are prone to eating some byte values if they are not correctly
set to binary mode.
—
Reply to this email directly, view it on GitHub
https://github.com/nanopb/nanopb/issues/829#issuecomment-1310575152, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AB5CZMID2EVTJRZH4TUPTGDWHUQKRANCNFSM6AAAAAAR4C3T2Y
.
You are receiving this because you authored the thread.Message ID:
@.***>

Use a debugger and check the buffer before you call pb_decode, and also check the size you pass to pb_istream_from_buffer().

Ok i will try that.
Ive been working on an stm32 microcontroller, without using a debugger.
Ive been printing over the serial connection or an SPI OLED display for
debugging. I will first check the buffer using these methods.
it might take me a bit to figure out how to use the debugger.
On Thu, Nov 10, 2022, 10:21 AM Petteri Aimonen @.***>
wrote:

Use a debugger and check the buffer before you call pb_decode, and also
check the size you pass to pb_istream_from_buffer().
—
Reply to this email directly, view it on GitHub
https://github.com/nanopb/nanopb/issues/829#issuecomment-1310711365, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AB5CZMM3PU7V25NNO7SNQPLWHU4JJANCNFSM6AAAAAAR4C3T2Y
.
You are receiving this because you authored the thread.Message ID:
@.***>

STM32 has pretty good debugger support, and in my experience it is easy enough to set up also. A cheap STM32 discovery board or similar is enough as the debugger adapter.
Even if the data is correct, the next step will be to start stepping into the code to see what is actually happening, so you'll need some way to run a debugger anyway. Often a reasonable alternative is also to compile your code for PC instead and debug there first.

I'm using an stm32f746zg nucleo board with a built-in st-link adapter.
But its taking me quite some time to understand how to set up the debugger.
Do you have any tutorial to suggest?
I'm still searching for one

Ok I have open-ocd --> arm-none-eabi-gdb --> ddd working now.  Time to start debugging

Ok I have some results.
I just went for it in raw command-line gdb because I couldnt get any GUI's to work.
Each time the     if (pc_.readable()) { loop is run, the correct char is read from the serial line.  However, the data is not properly stored in stream_buffer.
For some reason, stream_buffer contains \n\032\b, and doesnt accumulate any more bytes, even when more than 3 bytes have been read from the stream.
Furthermore, the actual message starts with \n\x1a\x08.
So something is going wrong with the line stream_buffer[bytes_read] = c;
For reference, I am calling receive_message as receive_message(buffer_MOTDrop,minig_MOTDrop_fields,&MOTDrop);
where the buffer is declared pb_byte_t buffer_MOTDrop[minig_MOTDrop_size];.  According to my .proto, minig_MOTDrop_size=256, so the buffer size should not be an issue.
Any ideas?
Thanks again for your help

If you are allocating the buffers on stack, make sure your stack is large enough to hold them.
Or you can make the buffers static instead, to allocate them in .bss section instead.
You can also debug instruction-by-instruction to see more clearly what is happening. In gdb, set display /i $pc to show current instruction and use stepi to step instruction-by-instruction (pressing enter will repeat it), info registers will show values in all registers.

I'm currently unable to reproduce this problem.


Clarifications to example readme documents

Regarding issue 1, I was working on this myself today. Here are some suggested clarifications.
(I have got the three simplest examples running on an azure VM with minikube, the ignored scripts/pj* auto build that)

Thank you! However, it's very hard for me to take in a 15-commit PR. Could you please squash all commits into a single commit and resubmit?
Also, I think we have been working on the same file at the same time. If you look at my latest commit perhaps it would already address some of your corrections.

By the way, you did not have have to create a new PR. If you do a push --force to the branch, github will automatically update the PR. Nifty!


Database Schema Design

Database Schema Design

As a developer
I need to design the database schema to store user data, product information, and interaction data,
So that the recommendation system can retrieve and process data efficiently.

Details and Assumptions

Use a relational database.
Optimize schema for read and write performance.
Ensure scalability.

Acceptance Criteria
**Given** a set of database tables,
**When** querying for user interactions,
**Then** the database should return results efficiently and accurately.

-- Users Table
CREATE TABLE Users (
user_id INT AUTO_INCREMENT PRIMARY KEY,
name VARCHAR(100),
email VARCHAR(100) UNIQUE,
registration_date DATE
);
-- Products Table
CREATE TABLE Products (
product_id INT AUTO_INCREMENT PRIMARY KEY,
name VARCHAR(100),
description TEXT,
category VARCHAR(50),
price DECIMAL(10, 2)
);
-- Interactions Table
CREATE TABLE Interactions (
interaction_id INT AUTO_INCREMENT PRIMARY KEY,
user_id INT,
product_id INT,
interaction_type VARCHAR(50),
interaction_date TIMESTAMP,
FOREIGN KEY (user_id) REFERENCES Users(user_id),
FOREIGN KEY (product_id) REFERENCES Products(product_id)
);
-- Recommendations Table
CREATE TABLE Recommendations (
recommendation_id INT AUTO_INCREMENT PRIMARY KEY,
product_id INT,
recommended_product_id INT,
recommendation_type ENUM('cross-sell', 'up-sell', 'accessory', 'customer_purchase'),
FOREIGN KEY (product_id) REFERENCES Products(product_id),
FOREIGN KEY (recommended_product_id) REFERENCES Products(recommended_product_id)
);


Add JSON Documentation

This issue is for tracking the requirements, issues and solutions in generating JSON for the gcloud-common Angular site.
Known Requirements

[ ] Extracting sphinx code examples and place them in methods[{examples:[]}].
[ ] Generate table of contents from sphinx toc. (index.rst).
[ ] Class linking should be added. i.e. support :class:gcloud.datastore._generated.query_pb2.Query, this is partially done through types.json.
[ ] Source code links (base on relative repo path. i.e. /google/cloud/datastore/client.py#L25).
[ ] "Getting Started" and "Guide" type pages need to be ported.
[ ] Ensure that snippets are pulled in properly from docs/*_snippets.py.
[ ] Turn on auto gcloud-common/site deployments.

Discussions

How much data is available in Writers https://groups.google.com/forum/#!topic/sphinx-users/-J1IOOBd-0g

Resources

http://www.arnebrodowski.de/blog/write-your-own-restructuredtext-writer.html
https://sourceforge.net/p/docutils/code/HEAD/tree/trunk/docutils/
https://github.com/sphinx-doc/sphinx
https://bitbucket.org/birkenfeld/sphinx-contrib

@daspecster Did you mean to use gcloud.datastore._generated.query_pb2.Query as an example? Those generated files don't have any docs.

Actually yes I did.
We would need to do one of two things with a reference to generated code.

Strip the directive and just not link it anywhere.
Just link it to that file/line in the repo (which is what I did/tried to do before since I generated JSON for everything under gcloud/, now google/cloud).

The bigger thing is generating types.json, I just used that one to illustrate this case.

I am :-1: on linking into the source. Those protoc generated files are not meant for human eyes and are very nasty to look at.

Sorry, I wasn't very clear on that.
Not linking to the source in this case(however that is a required feature for documented modules,classes,methods,params,variables.
I was generating a JSON file for the _generated files as well since I was doing all python files under gcloud at the time. So the JSON representation was there, but not linked in the TOC. So it just means when there's a reference to one of those in a docstring, the :class: directive works as expected.

@lukesneeringer is this project still in some long-term "TODO"?

I had no idea this was even a thing.

Denied.


message_create_time inconsistent and ambiguous in Experiment

Creating an issue to avoid side tracking PR636, https://github.com/ga4gh/schemas/pull/636#discussion_r67723063

Why message_create_time here? The work message seems ambiguous, i.e., does this refer to the individual message objects as they are being sent to the client, or to the 'record' that is stored on the server? I'm guessing that this is because of a clash with run_time? I would have thought it would be better to just use created and updated just for the sake of being consistent with all the other objects (and change them all in one go to something more specific like creation_timestamp, update_timestamp, if/when this is done).

also the use of the term `message_' is confusing.  It needs to be abstracted.
generally, all of the time fields are not documented clearly.  We need to clean this
up.
Jerome Kelleher<EMAIL_ADDRESS>writes:

Creating an issue to avoid side tracking PR636, #636 (comment)
Why message_create_time here? The work message seems ambiguous, i.e., does
this refer to the individual message objects as they are being sent to the
client, or to the 'record' that is stored on the server? I'm guessing that
this is because of a clash with run_time? I would have thought it would be
better to just use created and updated just for the sake of being
consistent with all the other objects (and change them all in one go to
something more specific like creation_timestamp, update_timestamp, if/when
this is done).

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.*

This was again mentioned in https://github.com/ga4gh/schemas/issues/682  and is being addressed in  https://github.com/ga4gh/schemas/pull/690. Closing.


Material system: load GLSL extension needed for Intel

This gets the material system running on Intel drivers. In a cursory test, I didn't see any glaring bugs.
I searched for "intel glsl wrong line number" and by chance found a thread about this extension.

Actually it doesn't help. I must have made some mistake when testing.


issue #1237 - stop adding duplicate SearchParameters with base Resource

I found that we were adding the Resource-level SearchParameters in two
different places and this was causing some inconsistent behavior.
In this option, I stop adding the Resource search-parameters at the end
of getFilteredBuiltinSearchParameters.
This effectively reverts us to the searchfilter behavior we had in our
DSTU2 implementation; if a filter is present for a given type, then the
common parameters must be explicitly listed as allowed search parameters
as well.
Also: removes redundant getSearchParameterByCodeIfPresent in favor of
getApplicableSearchParametersMap(resourceType).get(code)
Signed-off-by: Lee Surprenant<EMAIL_ADDRESS>

closed in favor of #1239