closed ticket #84. showing error message on page for overlapping even…

…ts. new bug discovered with ticket #104 made to resolve it on backend
Pull Request Template
1. Targeted Issue
Return error message for overlapping event schedule
Ticket #84
2. Overview of Solution
Set .catch in axios request in TimeSlot.js with error.response.data
3. Tools Used
React and MUI
4. Testing Strategy
Trial and error with monitoring inspect tools, Network and Console.
5. Future Implications
We now display an error message to let the mentor know they have potential scheduling conflicts. Also discovered new bug and wrote ticket #104 on trello
6. Screenshots
(A picture is worth a thousand words. Include screenshots of your changes if relevant.)
7. Code Reviewers
(Who else is in the loop? Mention your team or other contributors here.)

Before you go...

[ ]  I have linked the relevant issue
[x]  I have provided an overview of my solution
[x]  I have shared the tools I've used
[ x]  I have outlined my testing strategy
[x ]  I have considered future implications of my changes
[ ]  I have included relevant screenshots
[ ]  I have kept my team in the loop

Big high-fives for your incredible pull request! We're thrilled to have you as a contributor! Our team will dig into your proposed changes and give you some feedback or merge them if they're good to go. Your contributions mean the world to us!

Need to fix a few more things. Error banner is just constant on page right now.


Issue: Bot sometimes logs GIFs and URLs as message updates

Occasionally when running this bot and sending a GIF or URL in the target server the bot will log this, even though nothing was manually updated. My best guess is that it sees what Discord does automatically (e.g. rendering the link to the GIF into the GIF itself or adding an embed for a URL) as a message update, and it simply sends the identical source and updated message as a message update log. A simple example was https://github.com, which renders in a text channel with an accompanying embed, where the Logger Bot logged a source message of https://github.com and an updated message of https://github.com as well. Any chance there's a simple fix for this?

I should emphasize that this is not universal; in fact, if I send the same message again (https://github.com), it seems that the message update log is no longer triggered, but some other URLs and GIFs trigger the log. Maybe that has to do with a caching process, I'm really not sure.

@mlhynfield As I already wrote in another open issue, I haven't updated this code for over half a year because I don't have time for it, so the code is quite outdated and may not work, you can fork it and fix it so that everything works as it should.


Paging for amd64, coroutines/fibers, and more

Jenkins test please

jenkins test please

jenkins test please

jenkins test please


Metadata Fetch Fails When Creating PR For Github Enterprise

Using dependabot-script, it successfully reads my repo, recognizes a dependency to update and goes to attempt the PR, where it fails to fetch the ref_names. It seems to be trying to access https://github.com/$REPO instead of the provided GITHUB_ENTERPRISE_HOSTNAME  used when fetching the project originally.
Traceback (most recent call last):
	8: from generic-update-script.rb:119:in `<main>'
	7: from generic-update-script.rb:119:in `each'
	6: from generic-update-script.rb:171:in `block in <main>'
	5: from /usr/local/lib/ruby/gems/2.6.0/gems/dependabot-common-0.107.5/lib/dependabot/pull_request_creator.rb:66:in `create'
	4: from /usr/local/lib/ruby/gems/2.6.0/gems/dependabot-common-0.107.5/lib/dependabot/pull_request_creator/github.rb:39:in `create'
	3: from /usr/local/lib/ruby/gems/2.6.0/gems/dependabot-common-0.107.5/lib/dependabot/pull_request_creator/github.rb:66:in `branch_exists?'
	2: from /usr/local/lib/ruby/gems/2.6.0/gems/dependabot-common-0.107.5/lib/dependabot/git_metadata_fetcher.rb:27:in `ref_names'
	1: from /usr/local/lib/ruby/gems/2.6.0/gems/dependabot-common-0.107.5/lib/dependabot/git_metadata_fetcher.rb:16:in `upload_pack'
/usr/local/lib/ruby/gems/2.6.0/gems/dependabot-common-0.107.5/lib/dependabot/git_metadata_fetcher.rb:51:in `fetch_upload_pack_for': The following git URLs could not be retrieved: https://github.com/$REPO(Dependabot::GitDependenciesNotReachable)
	6: from generic-update-script.rb:119:in `<main>'
	5: from generic-update-script.rb:119:in `each'
	4: from generic-update-script.rb:171:in `block in <main>'
	3: from /usr/local/lib/ruby/gems/2.6.0/gems/dependabot-common-0.107.5/lib/dependabot/pull_request_creator.rb:66:in `create'
	2: from /usr/local/lib/ruby/gems/2.6.0/gems/dependabot-common-0.107.5/lib/dependabot/pull_request_creator/github.rb:39:in `create'
	1: from /usr/local/lib/ruby/gems/2.6.0/gems/dependabot-common-0.107.5/lib/dependabot/pull_request_creator/github.rb:65:in `branch_exists?'
/usr/local/lib/ruby/gems/2.6.0/gems/dependabot-common-0.107.5/lib/dependabot/pull_request_creator/github.rb:68:in `rescue in branch_exists?': Unexpected git error! (RuntimeError)

Notice the line URLs could not be retrieved: https://github.com/$REPO
Debugging from dependabot-script, all the information (including @files) looks correct in the object created here but it fails to use the enterprise repo here
Is there something I should be doing differently to get it past this point?

Note: this seems to be a recent-ish regression. It broke somewhere between 0.106.30 and 0.106.47

Fixed in https://github.com/dependabot/dependabot-core/commit/83adb157ef2ac70d737794679c0f3ff69e479c13. Thanks for the heads up!


Implementation of Javascript aggregator for DQL

Hi I am working on the DQL and I am facing difficulty with how to implement the javascript aggregator for the druid query language.

I've prepared an update in order add support for Javascript aggregators in Scruid.
To pass the tests, it requires a Druid server with Javascript support enabled. @Fokko I've made a PR in https://github.com/Fokko/docker-druid/pull/3 in order to have a docker image with Javascript support enabled.

My pleasure @anskarl. Looking forward to the aggregator.

@anskarl Is there the implementation of javascript post aggregator for DQL?

@anskarl Is there the implementation of javascript post aggregator for DQL?

@utk14 the implementation is in the pull request https://github.com/ing-bank/scruid/pull/77
An example can be found in docs https://github.com/anskarl/scruid/blob/feature/js_aggregation/docs/dql.md#javascript-aggregator

@anskarl If I am not wrong your PR is supposed to be the JavascriptAggregator..
I wanted to know whether there is any implementation for javascriptPostAggregator.
I want to implement something similar to Druid's javascript post aggregator (https://druid.apache.org/docs/latest/querying/post-aggregations.html):
{
  "type": "javascript",
  "name": "absPercent",
  "fieldNames": ["delta", "total"],
  "function": "function(delta, total) { return 100 * Math.abs(delta) / total; }"
}

You are right @utk14, I've updated the PR with Javascript post-aggregator in DQL.

@anskarl Thanks for the help. Is your PR merged?

I think the issue can be closed, since the corresponding PR has been merged to the master branch

Thank you for the update @anskarl. By any chance @utk14, did you find an opportunity to test the aggregator?

@Fokko Yeah It worked perfectly.. Please let me know when will you release the maven dependency with the latest changes?

Awesome, great to hear. @mmartina any plan for a release? :-)

Hi @Fokko any updates when you are planning to release the maven version?

v2.3.0 released.


Certification "Airflow fundamentals" for Pankaj

I have completed the certification with a score of 66/75 (88%). Took a look at the questions I answered wrong and am clear with those now.


nc_def_var_bzip2 is not documented

This is in the current 4.9.1 documentation.  I don't see bzip2 in the list.

Right here now: https://docs.unidata.ucar.edu/netcdf-c/current/netcdf__filter_8h.html#a6f0038befa623732c8d1578c2e47736b
This issue should be closed.


Typo "@model"→"@model"

Italic with escape characters
https://docs.microsoft.com/en-us/aspnet/whitepapers/mvc3-release-notes

Duplicate #241
closed #244


Added Hover effect for "Simulator" icon

Fixes #
Added hover effect
Describe the changes you have made in this PR -
All other nav items have hover effect. The simulator icon also needs a nice hover effect as it is the most significant item.
Added hover effect as shown below
###Before

###After

Coverage remained the same at 84.266% when pulling 6e8ec0cfdd7e0e042c2b576c2d1a736be6102ae7 on Vishruth-S:master into 1145c3acff926c1f7f309f6a581c18bac0e3c070 on CircuitVerse:master.

@nitin10s @aayush-05 approve either this or #1109 and we can merge.

This looks good to me, @Vishruth-S can you use sass pseudo selectors ?

@aayush-05 Replaced with Sass pseudo selectors as requested. kindly review

Can you add a transition like #1105

@nitin10s Done as requested. Kindly confirm if working as expected

I was asking for hover effects like in the other PR, this one makes the opacity 0 on hover. Pls see and fix @tachyons don't merge please.

@nitin10s I am sorry i didn't understand what u meant..
This is the working of the changes i made

Could you please specify what exactly to be modified here

@nitin10s Made changes as requested, kindly review

Can you update the description with new screenshots/video ?

@nitin10s shall we close this one, I am planning to implement it myself; since this fix was having inconsistencies ?

@tachyons can be closed; fixed here >> #1451


Modify pipeline to accept GPU device id

Currently, pipelines have a GPU flag to enable/disable. This parameter should also support integer device numbers. This will allow handling multi-GPU systems.

Hi @davidmezzetti,
I would like to try this. shall I create a draft PR?

Not sure if this was still being work but a major refactor is in the works that added this feature. Thank you for volunteering regardless!


[TikTok] Support for Photos

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm requesting a site-specific feature
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[ ] I've read about sharing account credentials and I'm willing to share it if required

Region
United States
Example URLs
https://www.tiktok.com/@chososfavhairtie/photo/7369836402751671585
https://www.tiktok.com/@itsevykins/photo/7367130722806648106
https://www.tiktok.com/t/ZPRK8TCAT/
Provide a description that is worded well enough to be understood
Photos will either fail to download because there is no extractor or only download the audio.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['-vU', '-o', 'tmp2/%(title)s.%(ext)s', 'https://www.tiktok.com/@chososfavhairtie/photo/7369836402751671585']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [ff0779267] (source)
[debug] Lazy loading extractors is disabled
[debug] Git HEAD: 34f60217e
[debug] Python 3.12.3 (CPython x86_64 64bit) - Linux-6.9.1-arch1-1-x86_64-with-glibc2.39 (OpenSSL 3.3.0 9 Apr 2024, glibc 2.39)
[debug] exe versions: ffmpeg 6.1.1 (setts), ffprobe 6.1.1
[debug] Optional libraries: sqlite3-3.45.3
[debug] Proxy map: {}
[debug] Request Handlers: urllib
[debug] Loaded 1804 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp
yt-dlp is up to date<EMAIL_ADDRESS>from yt-dlp/yt-dlp)
[generic] Extracting URL: https://www.tiktok.com/@chososfavhairtie/photo/7369836402751671585
[generic]<PHONE_NUMBER>751671585: Downloading webpage
WARNING: [generic] Falling back on generic information extractor
[generic]<PHONE_NUMBER>751671585: Extracting information
[debug] Looking for embeds
ERROR: Unsupported URL: https://www.tiktok.com/@chososfavhairtie/photo/7369836402751671585
Traceback (most recent call last):
  File "/home/wtromary/dev/yt-dlp/yt_dlp/YoutubeDL.py", line 1606, in wrapper
    return func(self, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/wtromary/dev/yt-dlp/yt_dlp/YoutubeDL.py", line 1741, in __extract_info
    ie_result = ie.extract(url)
                ^^^^^^^^^^^^^^^
  File "/home/wtromary/dev/yt-dlp/yt_dlp/extractor/common.py", line 734, in extract
    ie_result = self._real_extract(url)
                ^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/wtromary/dev/yt-dlp/yt_dlp/extractor/generic.py", line 2514, in _real_extract
    raise UnsupportedError(url)
yt_dlp.utils.UnsupportedError: Unsupported URL: https://www.tiktok.com/@chososfavhairtie/photo/7369836402751671585

I can work on this one too if you would like to assign me

Downloading photos is out-of-scope
Duplicate of #6168, #6217, #8360
Look into gallery-dl


Add provider static instance

Create branch: issue-16-add-provider-static-instance from develop.

Create branch: issue-16-add-provider-static-instance from develop.


Version numbering.

How do we go from:
4.1.0
4.1.1
4.1.2
4.1.3 to 3.4.0???

there are 2 active branches, v3 and v4.
loads of sites still use v3, so updates have been made to it over the years.
3.4.0 will be the last release of the v3 branch, and all future development will focus on v4 and v5.
If you have a new site, it is highly recommended to go with v4.

Ah, OK. Got you. Sorry wasn't aware of that.
I have my own reporter that tells me when many dev tools & libs update so this reported that it had gone from 4.1.3 to 3.4.0.
Thank you for your very prompt reply. All good now.

Great.
You can close the issue then.


Needs clarification on whether Power BI Embedded supports it or not

This page is unclear on whether the personalize visuals feature works or not in Power BI Embedded, within an ISV application.  Would be great if that could be clarified!  Thanks

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

ID: 615021e0-5504-046d-d6ab-c2d8dee0656f
Version Independent ID: d9c6ad5a-139f-716e-eb59-473a1f8d73de
Content: Let users personalize visuals in a report - Power BI
Content Source: powerbi-docs/create-reports/power-bi-personalize-visuals.md
Service: powerbi
Sub-service: powerbi-service
GitHub Login: @maggiesMSFT
Microsoft Alias: maggies

Thanks for your comment, @chrisweis. This issue has been assigned to the doc author for review.


Consistency on commands, caps, --flags and defaults.

The look and feel for the commands needs some TLC
for example mode [conformance]
e2e-focus [Conformance]
...
/cc @chuckha @liztio

I don't think there's a good way to resolve that particular issue. It makes sense for modes to be lowercase, because that sort of argument usually is, but e2e-focus has to be capitalised because that's how Ginko works.

Prior art for capitalisation:
  -o, --output='': Output format. One of: json|yaml|wide|name|custom-columns=...|custom-columns-file=...|go-template=...|go-template-file=...|jsonpath=...|jsonpath-file=... See custom columns [http://kubernetes.io/docs/user-guide/kubectl-overview/#custom-columns], golang template [http://golang.org/pkg/text/template/#pkg-overview] and jsonpath template [http://kubernetes.io/docs/user-guide/jsonpath].
Counter examples:
--session-affinity='': If non-empty, set the session affinity for the service to this; legal values: 'None', 'ClientIP'

the defaults presented could be Cammel Cased but internally everything is tolower'd.


zstd Compression Does not work with the Consumer

We recently switched over to using zstd and have some monitoring, and other tools that use this library. After updating to 2.0.2 we where disappointed to find out that it doesn't work with our data.
The problem seems to be that no matter what version you configure in the consumer it only sends FetchRequest_V4 as per this logic and FetchRequest_v10 is required for zstd and your own note
Sending FetchRequest_V4 against a topic that has zstd data returns this error:
DEBUG:kafka.protocol.parser:Processing response FetchResponse_v4
DEBUG:kafka.conn:<BrokerConnection node_id=0 host=localhost:9092 <connected> [IPv4 ('<IP_ADDRESS>', 9092)]> Response 4302 (1.6283988952636719 ms): FetchResponse_v4(throttle_time_ms=0, topics=[(topics='foobar', partitions=[(partition=0, error_code=76, highwater_offset=-1, last_stable_offset=-1, aborted_transactions=NULL, message_set=b'')])])

Namely the error_code=76 which is UNSUPPORTED_COMPRESSION_TYPE

do you accept a PR?

I do! I've been busy these days so I'm not actively writing or reviewing code. Once things calm down, I'll get back on the horse. 😀 Please feel free to open a PR though, if it's small, I'll merge it!

do you mind I skip other request version 4-9 and implement 10 directly on supported kafka version？

Go for it!

sorry, I just find that confluent-kafka-python support this, I'll just choose confluent-kafka-python....

looks like to support kafka v10， we'll need to support session first…

OK, good new is, session is optional feature and can be omitted.

https://github.com/wbarnha/kafka-python-ng/pull/200
I'm not sure it's a big or small PR, but it would works fine to send basic v10 request.
New feature can be omitted by sending zero value.


News customdata (has clicks) and news setorigin

Fix the issues and then I will merge. You are close!

Do I need to close and repush a new pull request?

No, commit the fixes, push and it will update the pull request

It's building but everything looks good I think.


How to set statical size?

Hello, How i can set width 1500px and height 250px in result and exportImage() ? I've tried many options, but nothing happen

@shakidev you can't change the size of the image returned by exportImage. That method returns image data that uses the size of the canvas element(s) used by wavesurfer. See wavesurfer.drawer.canvases array.


Pipelines enhancements

Some enhancements to your PR at https://github.com/MiloszKrajewski/K4os.Compression.LZ4/pull/74
See commits list for punch list of improvements.

BTW, props to you for being able to build. This repo has no contribution.md, and it doesn't seem to use msbuild to build (using 'fake' instead?) and paket instead of nuget. I only got by in making these changes by unloading most of the other projects in the solution.

Also, most of your new unit tests were failing even before I made changes. I don't know why or if you expected that.

Hi @AArnott, thanks for the changes. The tests should all pass. One issue could be if the "hello.bin" file is cloned with a wrong newline character. I have had that issue before. The content of that file should be a "Hello World\n". I tried renaming the file from hello.txt to hello.bin to avoid this, but maybe that could be it.

Ahh, I can now see from the commit message why you set "result = default" after advancing the reader. That is a good idea, but we need to save the IsCompleted flag before advancing then.

If you need specific line endings, add or edit a .gitattributes file. This ought to work:
*.bin -text

My next push will include the .gitattributes file, which I've confirmed does solve the problem.

BTW, I added an .editorconfig file because I noticed this repo uses tabs but your new code uses spaces. The new file will help avoid mistakes like this. You may want to run the Format Document command on each of your new files to apply the file's settings and switch to tabs.

All tests pass now.

This is really 👍 I hope that we can get this merged nicely into upstream. I made a nit comment about the unsafe code. After that I think that we are ready to merge.
I am not a stream expert, but would it be relative straight forward to make the plumbing between this and the current stream implementation? The reason that I ask is that by creating that now as a prototype in its own pr, we may be able to inherit a set of tests that could increase the confidence in this code. That should then be merged later after this have been battle tested in the wild 😜

I only saw one comment from you about unsafe code, and I resolved it. The unsafe code has been moved to the extension methods class that already (necessarily) had unsafe code in it. But I guess I left behind AllowUnsafeBlocks=true in the project file. I just pushed a commit to address that.


Some site's it works on some sites it dont.

Here is the error log
`ERROR:root:Error executing Cloudflare IUAM Javascript. Cloudflare may have changed their technique, or there may be a bug in the script.
Please read https://github.com/Anorov/cloudflare-scrape#updates, then file a bug report at https://github.com/Anorov/cloudflare-scrape/issues.
Traceback (most recent call last):
File "/root/tester/test.py", line 13, in
status_code = scraper.get(url).status_code
File "/usr/lib/python3/dist-packages/requests/sessions.py", line 501, in get
return self.request('GET', url, **kwargs)
File "/usr/local/lib/python3.5/dist-packages/cfscrape/init.py", line 47, in request
return self.solve_cf_challenge(resp, **kwargs)
File "/usr/local/lib/python3.5/dist-packages/cfscrape/init.py", line 77, in solve_cf_challenge
params["jschl_answer"] = str(self.solve_challenge(body) + len(domain))
File "/usr/local/lib/python3.5/dist-packages/cfscrape/init.py", line 120, in solve_challenge
result = node.exec_(js)
File "/usr/local/lib/python3.5/dist-packages/execjs/abstract_runtime.py", line 17, in exec
return self.compile('', cwd=cwd).exec_(source)
File "/usr/local/lib/python3.5/dist-packages/execjs/abstract_runtime_context.py", line 18, in exec
return self.exec(source)
File "/usr/local/lib/python3.5/dist-packages/execjs/_external_runtime.py", line 87, in exec
output = self._exec_with_pipe(source)
File "/usr/local/lib/python3.5/dist-packages/execjs/_external_runtime.py", line 108, in _exec_with_pipe
self._fail_on_non_zero_status(ret, stdoutdata, stderrdata)
File "/usr/local/lib/python3.5/dist-packages/execjs/_external_runtime.py", line 134, in _fail_on_non_zero_status
raise ProcessExitedWithNonZeroStatus(status=status, stdout=stdoutdata, stderr=stderrdata)
execjs._exceptions.ProcessExitedWithNonZeroStatus: (1, '', '')`

Could you share the website you're trying to scrap and if possible, then some code?

This is my code:
import cfscrape
import sys
import requests
from requests_toolbelt.adapters.source import SourceAddressAdapter
url = sys.argv[1]
#scraper = cfscrape.create_scraper()  # returns a CloudflareScraper instance
scraper = cfscrape.create_scraper()
scraper.mount('http://', SourceAddressAdapter(('myextraip')))
scraper.mount('https://', SourceAddressAdapter(('myextraip')))
status_code = scraper.get(url).status_code
if status_code == 403:
print("200")
if status_code == 200:
print("200")
it seems to be fixed now, will test it out to find a other domain with that error.

I think i found the problem, when i use sudo -u www-data python3 test.py linkhere it gives that error

Well, if the issue has been solved and it doesn't look like it's not the issue with the script, could you close this issue?

Looks like it is fixed with change the script to www-data group. :)


request new icon

Hi teams,
Thanks for your amazing icons.
But I hope you can add more icon such as handshake or something similar with the deal meaning.

Definitely see that being useful! We'll see what we can do.

I have a few icons to request, some of which I think are mandatory for icons libraries.

Bold

Italic

Underline

Strikethrough/Line through

arrow-line-middle-vertical

arrow-line-middle-horizontal

I have a few icons to request, some of which I think are mandatory for icons libraries.

These icons are already designed and will be in the next release! Next time start a new issue with a separate request. Moved to #9.

Handshake has been added to all libraries as of v1.1.0 🤝


[Enhancement]: Add a button to return to the home page

Do you want to have the enhancement of existing game ? 😀 Describe yourself..
Game enhancement :
There should be a button if the user decides in between to back to the home screen and readjust the settings.
Describe the solution you'd like
Solution steps
I can add a button which will take you back to the home page.
Select program in which you are contributing
Other
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hi,
Can you please assign this issue to me? I'm contributing under GSSOC'24 program.

Please assign this issue to me. I can add a button written exit on it which will take yes or no as input to return to home page or not.

Hi,
I am Kartik and I want to work on issue #23, which is adding a button to return to the home page
I will locate the HTML file where you want to add the button( index.html) or where the home page content is defined and I will write relative path to your home page.
Here is Linkedln https://www.linkedin.com/in/itskartikgupta/

Hi,
I want to work on this issue. Can you please assign it to me.

Hi,
I would like to work on this issue please assign this task to me

Hello, I'm a GSSOC'24 contributor
I would like to work on the same. Can you please assign this to me.

@kumarharshh @kvdevlopment @Harshdev098 @anythingkartik @AshutoshKumar0206 Please raise any other issue, because someone is already working on it.
If you have any doubt you can directly send message on our discord (https://discord.gg/rZb46cCMmK), we have created seprate chennals for all projects...
Thank you :)


Proposal: Add .joinInner and .joinLateral methods

It would be good to implement most used types of JOINs. As for me INNER JOIN and JOIN LATERAL are enough for start.

Maybe you mean Proposal not Proposition.

Proposition:

A suggested scheme or plan of action, especially in a business context. (Google definition)

Proposal:

A plan or suggestion, especially a formal or written one, put forward for consideration by others.  (Google definition)
The action of proposing a plan or suggestion. (Google definition)

Maybe the use of either one wasn't wrong and conveyed the intention clearly. :-)


feat: Display external links in iframes

After many wasted hours I realised that there is actually no way to check whether an iframe correctly loads, because:

If the iframe and the iframe's parent document are Same Origin, returns a Document (that is, the active document in the inline frame's nested browsing context), else returns null.

This means sometimes we won't be able to load external pages within an iframe. Unless there is a solution I'm not aware of.

We'll not be implementing this any longer. Closing for now


Generate Agent from config instead of creating new on server initialization

Currently, there is no real way to use an existing private key or external agent as a root agent in an atomic server. In other words, it's not possible to grant rights to any other user except for the generated user.
This can be fixed by parsing ~/.config/atomic/config.toml upon initialization.

Required some new functions for regenerating Agents and Public keys.

Required some new functions for regenerating Agents and Public keys.


chore: bdd-sbrp-import-1590506353 to 0.0.1

chore: Promote bdd-sbrp-import-1590506353 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


[Swift] Update for additional EqualLoc param on PatternBindingDecl::create

https://github.com/apple/swift/pull/17700 introduces a new parameter for PatternBindingDecl::create that takes the source location of the equals '=' token. Update the existing call sites to handle this.

apple/swift#17700
@swift-ci Please test


Gulp is not seeing changes made in LESS import files

We have 2 options, either touch all ''main' files using the changed import files, either do a rebuild but here many changes in the solution and commit pb later...
This seems to be a general Gulp Pb with import files.

Duplicate of #5849 (also opened by you).


Implement new rasterization mode using opacity compensation factor

See previous threads here: https://github.com/nerfstudio-project/gsplat/pull/117
The PR must be merged after this PR (version bump of gsplat lib, https://github.com/nerfstudio-project/gsplat/pull/124)

truck dataset:
Left = Using --pipeline.model.rasterize-mode 'standard'
PSNR = 26,19 dB
Right = using Splatfacto v1.0.1, so it don't use this rasterization.
PSNR = 25,97 dB

Currently only nerfstudio viewer is supported to display the results correctly. Most other viewers which were designed for the classic mode must accommodate the same change to show the right the results of this new feature.

Currently only nerfstudio viewer is supported to display the results correctly. Most other viewers which were designed for the classic mode must accommodate the same change to show the right the results of this new feature.

for truck and kitchen dataset that I have shown before, it was a frame from video which come from ns-render result. So, it does not screenshot the 3DGS Web viewer.
for playroom dataset, yes it is the screenshot from mkellogg 3D WebGL Gaussian Splat.

The new mode name is called antialiased

IMHO this should become the default, because:

"antialiased" hold a closer-to-reality physical representation of the scene
it doesn't hurt eval results when eval images are at the same distance as train images (there's even a slight improvement)
it would certainly improve eval results for images that are closer or farther

Note that the best test data for this is to use something with a fine or regular texture (eg coarse-thread fabric, things with grids).
As we are getting closer to a physical representation, I hope that my proposed change on working in linear space rather than gamma-compressed (https://github.com/nerfstudio-project/nerfstudio/issues/925) will have a visible impact some day.

The fact that third-party viewers don't render 2D Gaussians correctly is their own issue (some code don't even have the +0.3 trick)

Is it the same rasterization than this solution ???
it has a viewer based on mkkellog's one...  (unfortunatly, unavailable in the repo)

@gonzalle It is not exactly the same implementation as mip-splatting, but is in similar spirit and simpler. I don't think you can use that viewer to display PLY exports from splatfacto.

@jb-ye Could you please clarify why the opacity compensation is not being applied to the rasterization of the depth image?
https://github.com/nerfstudio-project/nerfstudio/blob/29ddd2ca3a33c43fb8d44b90d2758aaba0436f57/nerfstudio/models/splatfacto.py#L814

@jinhwanlazy Nice catch. I think you are right, this is a bug we have to fix. Could you send out a PR to fix it?


Notify on startup

Have the bot send an "I'm awake" message on startup.

@all-contributors please add @solenberg for ideas

We should look into utilizing the socket.io streamStarted event to do this.


Flow type references missing in docs

Hi, guys,
I see some flow type references missing in docs like this RgbaColor.

I'm playing with Flow and Documentation on another project and also faced this problem, but I finally managed to solve it.
Would a PR be welcome?

absolutely! yes please

Whoa I would love to solve this! Awesome!!


error: cannot find symbol class DateTimeView

Hi, I'm having this strange error show up. I've put your DrawGreen sample class into a new sample project, and am calling
Probe.deploy(this, new DrawGreen(), new Filter.ViewId(R.id.start_button));
on the onCreate of my Activity, just before the super.onCreate(). R.id.start_button is just the Button class.
I dug a little and it seems like this class is getting generated:
public final class ProbeProxy$android_widget_DateTimeView extends DateTimeView
    implements InterceptableView {
...
}

But it looks like it's unable to find DateTimeView.
I'm on Android Studio 2.1.1, compileSdkVersion 23.

+1 Having the same issue!

Though not nice, you can work aronud the problem by adding this to your build.gradle file:
     gradle.taskGraph.whenReady {
         probeDebugViews << {
             String path = buildDir.absolutePath + '/generated/source/probe/debug/'
             path += variant.mergedFlavor.applicationId.replace('.', '/')
             path += '/probe'
             File targetFolder = new File(path)
             File file = new File(targetFolder, 'ProbeProxy$android_widget_DateTimeView.java')
             println file
             file.delete()
         }
     }

It deletes the generated Java file. I added this directly below the probe {...} entry, but anywhere at the top level should work.


Split EmbeddingComposite into FindEmbeddingComposite and ApplyEmbeddingComposite

A user may want to only use the part of the EmbeddingComposite layer that handles seamlessly embedding and unembedding, but not calculate a new embedding each time. They may already have an embedding or they may want to use another tool (other than minorminer) to calculate their embedding.
As well, this logic is duplicated in VirtualGraph. Having the embed --> sample --> unembed workflow in one place would make it easier to update in the future.

Closed by #69


Support for reading exports map to determine entrypoints

Search Terms

exports map, export map, package exports
Problem

We're currently already listing all modules we export in the exports property in our package.json (Node docs, Webpack docs). It's great that since v0.20.0, TypeDoc now allows you to just specify the entry points to your package, but that means we're keeping two lists of those entry points.
Suggested Solution

Add an option to tell TypeDoc to look at the exports property in our package.json, and use the values in that object as the entry points. (And I suppose it could make sense to have it fall back to the module and then the main fields as well in case it's not present?)

I want to wait on this until that field is out of "Experimental" stability for Node, seems like a good idea though!

This is also tricky in that the "exports" likely don't match to exactly what we need to get... That map describes the output files, but typedoc really needs an equivalent one targeting the source directory containing TS. outDir could really mess with this...

Oh hmm, you're right... Duplication it is then, I'm afraid :/


Merged across all the patches to d3-zoom incrementally

In theory this should fix https://github.com/etiennecrb/d3-xyzoom/issues/3

@etiennecrb would you be able to publish this latest version to npm?

Sorry, I forgot! It's done.

@etiennecrb thank you!


Image preprocessing

Hello,
Thank you for your excellent work. I'm interested in understanding your data preprocessing methodology, particularly how you convert the original dataset into JPEG files. We're developing a benchmark dataset intended to integrate seamlessly with the SatlasPretrain model.
Our current process involves downloading images from Google Earth Engine, where the original pixel values for Sentinel-2 data range from 0 to 60,000. In light of this, I have two specific questions:

Which platform do you use to access the Sentinel-2 images?
What preprocessing steps do you take to convert the original values to the 0-255 range?

Thank you for your time and assistance.

We access the Sentinel-2 data from https://cloud.google.com/storage/docs/public-datasets/sentinel-2 (on free Google Cloud Storage bucket) and the normalization is detailed at https://github.com/allenai/satlas/blob/main/Normalization.md#sentinel-2-images


CI: run mypy in full env

[x] Closes #4881
[x] Tests added
[x] Passes pre-commit run --all-files
[ ] User visible changes (including notable bug fixes) are documented in whats-new.rst

I only added one run for py3.8 latest. To be entirely sure we could also check the typing py37-min-all-deps but it feels like overkill...

Ok, looks good - the failure is expected - see #4878.

the workflow looks good to me, but it should probably be possible to skip it using [skip-ci]

Thanks, @mathause, looks good to me.
I wonder if we should move this into linting? That way, running the linting on autoupdate PRs would be easier. That should probably be a new PR, though, so I'm merging.

Thanks - just for reference - this is currently unconditionally inactive & should be activated in #4878


Third-party apps / extensions: fluentcms-file

Thanks for django-fluent-[contents|pages], here is my small contribution https://github.com/bashu/fluentcms-file
PS: fluentcms-teaser and fluentcms-filer will be ready soon =)

Thanks!


[BUG]: No desktop during install, and also after install.

Athena OS version
Athena OS "Polias" v23.06.03
Are you using baremetal or virtual machine?

[X] Baremetal

[ ] VirtualBox Virtual Machine

[ ] VMware Virtual Machine

[X] ProxMox Virtual Machine

Kernel Version
No response
Other Software
No response
Repro Steps

Tested ISO boot and post-install boot in ProxMox with PCI-Passthrough
Tested ISO boot on MacBook Air 7,2
Tested ISO boot on MacBook Pro 11,3
Tested ISO boot on baremetal intel i7 10700

Expected Behavior
Window manager should load.
Actual Behavior
All tests resulted in the same error, upon user session load, it halts for sometime, and then boots into a gray screen with an error saying "something went wrong, please log out and log back in again.", you do get the welcome screen and can access the different links, but mouse isn't visible, also calamares works, i tried both login managers thinking it could be the cause, but didn't work, installed i3-wm manually and that worked.
Diagnostic Logs
No response

@AdhamNasr Hey! I am aware of the issue. It was caused by a package related to GDM that created a broken symbolic link instead of a file.
I already fixed and just now I finished to upload the new ISO. Please, download it again from https://sourceforge.net/projects/athena-iso/files/v23.06.03/athena-2023.06.03-x86_64.iso/download
I didn't still test it but it should work. Let me know.

Oh wow, that was quick!! Thanks for the update, I'll download the new one now :)

All good now, installer worked fine on MacBook Air.
Keep up the good work :)

All good now, installer worked fine on MacBook Air. Keep up the good work :)

Thank you mate. MacBook Air with which CPU? Intel or ARM?

All good now, installer worked fine on MacBook Air. Keep up the good work :)

Thank you mate. MacBook Air with which CPU? Intel or ARM?

You're welcome :)
It has an intel i5-5250U


#2482: Bio: exception on package loading initBio, add test

[ ] add test

This issue has been mirrored in Jira: https://reddata.atlassian.net/browse/GROK-14885

This issue has been mirrored in Jira: https://reddata.atlassian.net/browse/GROK-16145


net-web WG meeting 1

Adds the meeting notes from the first Net Web WG meeting. Thanks!

This was taken from paper as-is. Merging!


Shared attribute between different product types

What I'm trying to achieve
The attribute couldn't be shared between different product type. Each product type must have its own attributes. How about making same attribute able to be shared by different product types?
I can create a productTypeA with size attribute (S, M, L). But if I would like to create another productTypeB with same size attribute, I have to create another size attribute, even they have same values.
Will there have any issue if we change the product_type and  product_variant_type of Attribute to ManyToMany fields?

PR #4351 reintroduces standalone attributes that can be shared between different product types.


css: remove the left hand column?

the pipeline/log view has a left hand dark blue column thats probably not required. Removing it would let us see a little bit more log output for long log lines...

I let this dark blue column to give me the opportunity in a future to create an "options" side block (like https://codepen.io/maximakymenko/pen/aboWJpX for example).
If you think we don't need it for the moment. I can easily remote it 😉


Assorted small bug fixes

#3752 Lint 'semicolons' and 'line length' options change together

Copy/paste typo, fixed

#3750 Exclude dotted names from Pascal and camelCase

added check that name doesn't have dots

#3749 Snippets should not multiply spaces

instead of not formatting operator snippets, do format but append space at the end

#3747 Check for updates... still visible in 2017

removed command

#3746 RMD should update blocks on execute command

force update on auto-sync option change, lazy update of text changed before running the command

#3729 Graphics device does not initialize in markdown session

race issue, callback must be initialized before starting the host

#3751 Indent in complex expressions should respect last line indent

improved analysis of current/incomplete statements + tests

:shipit:


Cachi2 empty pkg managers

When users specify empty list as pkg_managers, only cloning should be
done. In this case usage of cachi2 must be skipped and required files
like env.json and bom.json must be created by OSBS.
Maintainers will complete the following section

[ ] Commit messages are descriptive enough
[ ] Code coverage from testing does not decrease and new code is covered
[ ] Python type annotations added to new code
[ ] JSON/YAML configuration changes are updated in the relevant schema
[ ] Changes to metadata also update the documentation for the metadata
[ ] Pull request has a link to an osbs-docs PR for user documentation updates
[ ] New feature can be disabled from a configuration file

updated, reworded

Rebased and removed DO NOT MERGE commit


Update readme.md

What Changed
Why
Todo:

[ ] Add tests
[ ] Add docs
[ ] Add yourself to contributors (run yarn contributors:add))

@all-contributors Please add @guptaatit94 for docs.

@all-contributors Please add @guptaatit94 for docs.

@all-contributors Please add @scanbhavin for docs.

@all-contributors Please add @scanbhavin for docs.


wrong result when using tween on "width" and calling "getBounds" in "update" loop

A bug in the API:
I have following incorrect working code.
create () {
this.g = this.game.add.graphics();
this.g.lineStyle(2,0xff0000, 1);
this.g.drawRect(50,50,150,150);
const tw = this.game.add.tween(this.g);
this.g.width = 1;
tw.to({width:150}, 1000);
tw.start();
},
update(){
this.g.getBounds();
}
After tween ends, I have wrong  "width", "scale.x", also "position.x" values. I found problem in the following. (Situation is presented in common and with not real values).
RAF  first loop
1)
Stage.Update -> g.getBounds() -> g._currentBounds.width = 1;
2)
Tweens.Update -> g.width = 2 -> g.scale.x = (2/g._currentBound.width) = 2
3)
postUpdate -> g.updateTransform() -> g.worldTransform.a = g.scale.x = 2.
RAF next loop
1)
Stage.Update -> g.getBounds() -> g._currentBounds.width = (worldTransform.a * width) = 2*150 =300
2)
Tweens.Update -> g.width = 3 -> g.scale.x = 3/300 = 0.01
3)
postUpdate -> g.updateTransform() -> g.worldTransform.a = g.scale.x = 0.01.
RAF next loop
1)
Stage.Update -> g.getBounds() -> g._currentBounds.width = (worldTransform.a * width) = 0.01 * 150 = 1.5
2)
Tweens.Update -> g.width = 4 -> g.scale.x = 4/1.5 = 2.66
3)
postUpdate -> g.updateTransform()-> g.worldTransform.a = g.scale.x = 2.66.

I think this is the correct result: https://codepen.io/samme/pen/paMEeG?editors=0010.
width === bounds.width * scale.x

height === bounds.height * scale.y

getBounds()
Retrieves the bounds of the graphic shape as a rectangle object

@samme No. It is wrong. Try without "render" and you will see.

Tween the scale instead: https://codepen.io/samme/pen/BYXrXd?editors=0010

@samme Thank you. I have solution. I solved it. Here I reported bug ("width" calculated based on "scale" and after it "scale" is calculated based on "width") to be fixed.

I see it now, thanks.


HDDS-6478. EC: Fix ISA-l load hadoop native lib UnsatisfiedLinkError

What changes were proposed in this pull request?
fix isa-l load hadoop native lib UnsatisfiedLinkError
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-6478
How was this patch tested?
ci

You could also remove {private long nativeCoder;} from AbstractNativeRawEncoder file as well. Thanks

ok，thank you very much


Add support for other HTML sources

E.g. Slim templates, Rails helpers.

Slim: the CLASS_LIST regex needs to find class: "..."
https://github.com/airblade/tcs/blob/f1b1c3085631d84ad0c38f02a942114f50f699eb/tcs#L16

It's impossible to support Slim's class shortcut because the . separating class names is the same as the . within Tailwind class names such as w-1.5.
∴ If you want to use tcs with Slim, you need to write out the class names using the class="..." attribute.

Maybe you could make special rules for \d+\.\d+ to be treated as one and also parse parentheses and not split class names inside those?

@bb As far as I'm aware Slim itself doesn't support class names containing ..  I know we discussed it previously over here while talking about Tailwind's arbitrary .

(I didn't mean to close this.)


Documentation on readthedocs

Updating version number 2.6.0
Adding readthedocs build with sphinx/breathe/exhale
Renames the doc folder to docs to standardize across rocm projects

Should be merge squashed

Please change the target to develop.

Could you solve the conflict?


Fix Node.js and Yarn installation in CI

Description
The GitHub workflow uses the engines from OSD incorrectly. This change fixes that.
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.

@AMoo-Miki Do we need to backport to 2.x?

@AMoo-Miki Do we need to backport to 2.x?

Yes please :)


[istio 1.4.6] : nil pointer evaluating interface {}.enabled

Hello
I am using istio 1.4.6 , while compling I am getting
executing "certmanager/templates/poddisruptionbudget.yaml" at <.Values.global.defaultPodDisruptionBudget.enabled>: nil pointer evaluating interface {}.enabled
role.yaml:4:7: executing "istio/charts/gateways/templates/role.yaml" at <($spec.sds) and (eq $spec.sds.enabled true)>: can't give argument to non-function $spec.sds
rolebindings.yaml" at <$spec.sds.enable>: nil pointer evaluating interface {}.enable
my sds flag is false and mtls flag to false and lobal.defaultPodDisruptionBudget.enabled = true

Hi, sorry this isn't working for you. Unfortunately, Istio 1.4 is no longer supported https://istio.io/latest/news/support/announcing-1.4-eol-final/ and this code is no longer present in our code base at all, so we won't be able to fix this. Thanks!


perf: move top of the execution stack instead of copy

Reduces time to run benchmark from ~22" to ~13.5" on my M1.

Tested and it works like a charm. Thank you!!!


Allow us to pass any argument to the Solidity compiler

Component
Forge
Describe the feature you would like
So that we don't need to wait 2 days for nightly to activate latest solc features in future.
Waiting 2 days is bad. Not blazing fast.
Some of us might want to bypass footgun protection.
Additional context
No response

triggering release with 0.8.24 rn
this feature is out of scope

Thanks for blazing fast response!


Typescript types error

"genshin-db": "^3.10.3"
node_modules/genshin-db/index.d.ts:99:2 - error TS1036: Statements are not allowed in ambient contexts.

99 };

Oops. You can remove the semicolon to get it working temporarily. I will fix this tomorrow. Thanks for catching this! Sorry for the inconvenience.

I made a pr to fix that https://github.com/theBowja/genshin-db/pull/53


generate .bvh

Is there an open source library that automatic capture of character action and generate the bvh file？

I tried, but even with manual retargeting, the animate is still out of control.

I tried, but even with manual retargeting, the animate is still out of control.

Can you tell me how you generate the bvh file

@ARDUJS what do you mean by 'automatically capture of character action'? What sort of character are you trying to get motion data from?
If you mean a video of a person, you can generate a BVH file directly from a video using a product like Rokoko or Deepmotion.
If you mean a stylized cartoon clip or something similar, extracting good skeletal motion from it is, to my knowledge, still an open research question.

@hjessmith is there an open source library to generate the .BVH from a video ? Because the 2 platforms that you mentioned (Rokoko, Deepmotion) seems not to be free, also you cannot automate the process using these 2 platforms.

Sorry, I'm unaware of any open source solutions.

@hjessmith is there an open source library to generate the .BVH from a video ? Because the 2 platforms that you mentioned (Rokoko, Deepmotion) seems not to be free, also you cannot automate the process using these 2 platforms.

https://github.com/HW140701/VideoTo3dPoseAndBvh
https://github.com/KevinLTT/video2bvh
I found these, but I haven't run them successfully yet.

@hjessmith is there an open source library to generate the .BVH from a video ? Because the 2 platforms that you mentioned (Rokoko, Deepmotion) seems not to be free, also you cannot automate the process using these 2 platforms.

https://github.com/HW140701/VideoTo3dPoseAndBvh https://github.com/KevinLTT/video2bvh
I found these, but I haven't run them successfully yet.

Thanks I've already tried both, they don't seem that are working, if somebody was able to run them, please share.

Now I download the action from https://www.mixamo.com/, and then export bvh in blender. The action can basically be displayed but the effect is not good. It should be that the manual configuration is not understood correctly. Let’s continue to try.

interested as well

If you mean a video of a person, you can generate a BVH file directly from a video using a product like Rokoko or Deepmotion.

I search the tutorial of Rokoko Studio， but still not able to find a way to import a video from my phone and export  a BVH file.. :(
i need to use all the Rokoko equipment to recording my motions, then i can export to BVH file.
is my understanding correct?

@professorLea take a look at this: https://www.rokoko.com/products/video


Consolidate common requirements to make dependabot happy

The unit and integration tests requirments files were both specifying pytest and that caused a conflict for dependabot.  This PR corrects that by adding yet another requirement file, requirements_common.txt.
In addition, the Makefile and .travis.yml were updated for the recent changes to tardigrade-ci.

The Travis job appears to be stuck.  I checked here:  https://app.travis-ci.com/github/plus3it/terraform-aws-org-new-account-iam-role and I think that's a previous job, not the current job.  The "bump" commit shows success, so I would expect the Travis job to show success as well (in the section below "Review required").

The Travis job appears to be stuck. I checked here: app.travis-ci.com/github/plus3it/terraform-aws-org-new-account-iam-role and I think that's a previous job, not the current job. The "bump" commit shows success, so I would expect the Travis job to show success as well (in the section below "Review required").

The most reliable place I've found to check to see if Travis even saw the request is the "requests" page. From the project's Travis page, select More options -> requests, https://app.travis-ci.com/github/plus3it/terraform-aws-org-new-account-iam-role/requests.
Since that doesn't show a request for the last commit, I'd assume it was just an issue where the webhook didn't fire properly or there was some other transient interruption between the GitHub and Travis systems. Did you try to amend the last commit and force-push, or otherwise create a new trigger for the build event?

The other two PRs are fine.  It's typically one of the three that has a problem and it rotates among them like a bad joke.


Update manage-data-sources.md

"Your collection has been moved" should be "Your data source has been moved". The data source is the object that is moved to the selected collection. The collection itself is not moved!
Confusion arises because the window in the previous step is called "Move collection". Apparently one and the same window is used for moving different asset types, so a generic name has been attached to it. It would have been clearer if the window had been called "Move data source" because you don't move the collection but the data source...

@linda33wj

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Merging per email from Whitney Henderson.


form-data error

I'm trying to use mailgun.js with nestjs.
I don't get any type error but when I try to use the client, i get error.
my code:
import formData from 'form-data';
import Mailgun from 'mailgun.js';
import { Injectable } from '@nestjs/common';

@Injectable()
export class MailgunService {
  constructor() {
    this.mailgun = new Mailgun(formData);
    this.mailgunClient = this.mailgun.client({
      username: 'api',
      key: "<my-key>",
    });
  }
  private mailgun: Mailgun;
  private mailgunClient: ReturnType<typeof this.mailgun.client>;

  async send() {
    await this.mailgunClient.messages
      .create('<my-domain>', {
        from: 'Excited User <mailgun@<my-domain>>',
        to: ['<target-email>'],
        subject: 'Hello',
        text: 'Testing some Mailgun awesomness!',
        html: '
Testing some Mailgun awesomness!
',
      })
      .catch(console.log);
  }
}

error:
/home/drhe/Documents/Code/Nestjs/ketabchi-02/node_modules/.pnpm/mailgun.js@8.0.6/node_modules/mailgun.js/webpack:/mailgun/lib/formDataBuilder.ts:30
      }, new this.FormDataConstructor());
         ^
TypeError: this.FormDataConstructor is not a constructor
    at e.createFormData (/home/drhe/Documents/Code/Nestjs/ketabchi-02/node_modules/.pnpm/mailgun.js@8.0.6/node_modules/mailgun.js/webpack:/mailgun/lib/formDataBuilder.ts:30:10)
    at e.postWithFD (/home/drhe/Documents/Code/Nestjs/ketabchi-02/node_modules/.pnpm/mailgun.js@8.0.6/node_modules/mailgun.js/webpack:/mailgun/lib/request.ts:161:43)
    at e.create (/home/drhe/Documents/Code/Nestjs/ketabchi-02/node_modules/.pnpm/mailgun.js@8.0.6/node_modules/mailgun.js/webpack:/mailgun/lib/messages.ts:59:25)
    at MailgunService.send (/home/drhe/Documents/Code/Nestjs/ketabchi-02/src/mailgun/mailgun.service.ts:19:8)
    at AuthService.signinLocal (/home/drhe/Documents/Code/Nestjs/ketabchi-02/src/auth/auth.service.ts:80:25)
    at /home/drhe/Documents/Code/Nestjs/ketabchi-02/node_modules/.pnpm/@nestjs+core@9.2.1_e6la6qvsclaae2becwjnmfvsuq/node_modules/@nestjs/core/router/router-execution-context.js:46:28
    at /home/drhe/Documents/Code/Nestjs/ketabchi-02/node_modules/.pnpm/@nestjs+core@9.2.1_e6la6qvsclaae2becwjnmfvsuq/node_modules/@nestjs/core/router/router-proxy.js:9:17

PS: I removed my api key and domain for its irrelevant
I don't even get runtime error until i call this send method.
package versions:
    "form-data": "^4.0.0",
    "mailgun.js": "^8.0.6",

node version: 16.19.0

I am getting the exact same error.  Does anyone have a working example of mailgun.js working with node.  mailgun-js works just fine but is deprecated.  I have the same package versions.

This can happen due to the formData import being invalid. Depending on your configuration (whether you are using TypeScript, ESM, plus the config options you use) you might need to change the import.
The import might look like:
import formData from 'form-data';
or
import * as formData from 'form-data';
or even...
import * as formDataExports from 'form-data';
// @ts-ignore
const formData = formDataExports.default as typeof formDataDefault;

This last one worked for me in getting ESM and TypeScript to play nicely.

well none of those work....
The first two generate the same error TypeError: this.FormDataConstructor is not a constructor
the last one work in a js file ;-).

Hello @Sina-Salahshour
I finally found time to investigate/reproduce the issue.
Sorry for not getting back to you sooner, It took some time to set up the nestjs app.
I was able to reproduce the problem and fix it using the @Polarisation suggestion by updating the import for the form-data package.
Please use import * as formData from 'form-data'; for the import.
Here is a wider code example:
import * as formData from 'form-data';
import Mailgun from 'mailgun.js';
import { Injectable } from '@nestjs/common';

@Injectable()
export class MailgunService {
  constructor() {
    this.mailgun = new Mailgun(formData);
    this.mailgunClient = this.mailgun.client({
      username: 'api',
      key: "<my-key>",
    });
  }
  private mailgun: Mailgun;
  private mailgunClient: ReturnType<typeof this.mailgun.client>;

  async send() {
    return this.mailgunClient.messages
      .create('<my-domain>', {
        from: 'Excited User <mailgun@<my-domain>>',
        to: ['<target-email>'],
        subject: 'Hello',
        text: 'Testing some Mailgun awesomness!',
        html: '
Testing some Mailgun awesomness!
',
      })
      .catch(console.log);
  }
}

Let me know if for some reason it doesn't work for you.

hi @olexandr-mazepa thank you for answering.
changing the import to import * as ... worked for me and the error got resolved.

Sina-Salahshour Nice to hear that issue is fixed.

Please note the issue still exists using svelte. To resolve it use
import FormData from 'form-data';


Mod auth basic cfg

Codecov Report

Merging #152 into develop will increase coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #152      +/-   ##
===========================================
+ Coverage    52.81%   52.82%   +0.01%
===========================================
  Files          217      217
  Lines        21573    21573
===========================================
+ Hits         11394    11397       +3
+ Misses        8885     8883       -2
+ Partials      1294     1293       -1

Impacted Files
Coverage Δ

...e_modules/mod_trust_clientip/mod_trust_clientip.go
42.85% <0%> (-3.3%)
:arrow_down:

bfe_websocket/server_conn.go
74.24% <0%> (+1.51%)
:arrow_up:

bfe_http2/writesched.go
80.91% <0%> (+1.52%)
:arrow_up:

bfe_spdy/server_write_sched.go
80% <0%> (+1.53%)
: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 19d9b6e...e0bdd22. Read the comment docs.


New Component gist - manage gist in CLI

run ./gist/gist help or just go to README
for more information
Pull Request Label:

[ ] Bug
[ ] New feature
[ ] Enhancement
[x] New component
[ ] Typo

Pull Request Checklist:

[x] Have you followed the guidelines for contributing?
[x] Have you checked that there aren't other open pull requests for the same fix or component?

Yes, but since most of commands in this tool require Github API token (with gist scope), the tests I made didn't cover its core utilities. Maybe there is a better way to do the test?
I promise this tool is really useful and user-friendly with prompts for toke validation and other stuffs, please give it a try!

I like this, it will most definitely make it to bash-snippets. I will need to take some time to read through it and review it before I can bring it in but overall it looks like it brings some nice functionality! Good work 👍

@alexanderepstein
Cool! Also, I made some improvement and README in latest commits.
Because I am new to bash, so if you find somewhere in code hard to read,  smells weird or violates some conventions, don't hesitate to ask me to do fix.

@alexanderepstein
Did some improvements.
Also, I add my api token from another Github account into test, so for now the tests should really meaningful for most of the sub-commands. But the weird thing is, all tests pass on my machine (Ubuntu 18.04), but test for gist new and gist edit never works on Travis build. Could you give it a try on your machine?

@alexanderepstein
Thanks for your feedback, I'll resolve/answer them in these two days

README is up to date. Now it has index at the top. Also includes newly-added feature like gist tag, gist lan or pipe feature.

@alexanderepstein
Did some changes/improvements in the last few weeks:

Refactor/Bug fix for gist tag and the way to process description which are stored in local
Update README
Mac compatibility
debug option for JSON parsing
For your latest review, you request to apply checkInternet() on every http request like other scripts.
But since all request goes to github.com in this script, I just removed checkInternet() and integrated its logic into http_method().

All your request are resolved, now you can review this PR again!

Thanks for this, hopefully I will release a new version of bash-snippets soon and this will be part of that release!

Cool! Thanks for the review.
The script I maintain on gist(which doesn't include Bash-Snippets stuff) is ahead of this PR with some refinement. I'll make another PR later to sync them.


Blender Cycles render not working. It's all gray. "GPU denoiser creation has failed."

I search in others issue post and there is no solution, anyone has the solution? I create this issue to let anyone put it.
Im using an AMD RX 580 and Blender 4.2

Try to use blender 4.0
It could help


[Strategic Goal] Determine the timeline for the surveys

Define timeline and set up deadlines for each of the activities:
General Member Survey

Define Questions
Review Survey questionnaire
Create a Form with the final version of the questionnaire
Form is Open
Form is Closed
Answers are reviewed by SC

Community Survey

Define Questions
Review Survey questionnaire
Create a Form with the final version of the questionnaire
Form is Open
Form is Closed
Answers are reviewed by SC

In 2021, the member survey was open from December 7th through December 20th. I think maybe we should have them both open at the dame time, but do them earlier to avoid overlapping with the holidays.
Here's my proposed timeline - what do people think?

September 1: Start project

September 1: begin working on defining questions  (see #207)
September 1: define plan for distributing and reminders for the survey (see #208)

October 15: Questions defined for both surveys (see #207)
October 26: SC & LF review of questions completed
October 31: Create both forms with final version of questionnaires
November 4: Steering committee approval of final questionnaires
November 7: Survey forms opens
November 7 and December 1: Distribute surveys and send reminders according to #208
December 1: Survey forms closes
December 1 - 9: Answers reviewed by Steering Committee & next steps determined

Thanks for putting this together @geekygirldawn timeline works fine for my side 🙂 jus one question, the timeline you have proposed is for both surveys, right? (e.g when starting the "begin working on defining questions" task, we should work on questions for the GM and the Community survey at the same time)

Yes, exactly! I thought it would be easier to do both at the same time, since I'm guessing some of the questions will be the same or similar :)

@anajsana Just a quick reminder that we have Sept 1 on the timeline to start working on these surveys. Our deadline for having the list of Questions is Oct 15, so I think we have some time. Should we start working on the questions right after OSSummit?

FYI : Starting a draft with the TODO Survey Questionnaire for 2022. I'll share via Steering Committee mailing list no later this week for review and additions 🙂 @geekygirldawn @vmbrasseur
After reviewing previous surveys, I'm planning to do a single survey and add a small section for TODO General Members, instead of creating two different ones (community and general members). This way people don't get overwhelmed with so many surveys (e.g TODO Steering Committee elections, satisfaction surveys, etc).
If you think we should do two separate surveys, please let me know!

I've just shared via SC Mailing list the draft for the TODO Satisfaction Questionnaire for 2022. When ready, please interested people review and/or suggest changes in the document or leave feedback on this GH issue: https://github.com/todogroup/governance/issues/207

+1 to doing this is one survey.
My gut feel is that this is way too long, so before I provide detailed feedback on the questions, can you get some feedback from the survey experts within the LF / CNCF about how many questions are too many? Maybe there are some things they can suggest to streamline the survey? Or maybe this is fine, I'm just not sure?

Thanks @geekygirldawn !
I don't have direct contact with survey experts within the LF / CNCF but I can ping @caniszczyk to chime in 👍


[Synapse] Generate Azure.Analytics.Synapse.Artifacts beta.11

API changes have been detected in this PR. You can review API changes here

API changes have been detected in this PR. You can review API changes here

API changes have been detected in this PR. You can review API changes here

Where is SparkConfigurationClient ?

API changes have been detected in this PR. You can review API changes here

API changes have been detected in this PR. You can review API changes here


ParseExactConverter for UpdatedAt field in Greeks is bad

Reading works fine, but writing produces invalid json. Json will look like:
"greeks": {
	"delta": 0.0,
	"gamma": 0.0,
	"theta": 0.0,
	"vega": 2E-05,
	"rho": 0.0,
	"phi": 0.0,
	"bid_iv": 0.0,
	"mid_iv": 0.9879,
	"ask_iv": 1.9758,
	"smv_vol": 0.479,
	"updated_at": 9 / 29 / 2020 7: 56: 01 PM
}

You could probably just use MillisecondsEpochConverter like for other fields, which works just fine.

Hello @tinytownsoftware, thank you for reaching out with this!
You were right, the UpdatedAt was producing an invalid JSON. However, the problem was happening because the WriteRawValue function was missing the double quotes to write the date as a string.
Nevertheless, I just published a new version of the NuGet Package (1.0.7) with the fix and this should be working now.
Thank you again and let me know if you see any other problems!


"Săpunari" is listed in the drop-down menu

Description:
"Săpunari" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Săpunari" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Săpunari, CĂLĂRAȘI (Lehliu)".


How to document variable number of arguments for method?

According to the PHPDoc v1 manual we used to be able to document a variable number of arguments like so:
/**
 * Sum list of numbers
 * @param int $numbers,...
 * @return int
 */
function sum(int ...$numbers): int
{
  return count($numbers) ? array_sum($numbers) : 0;
}

But this causes these Compilation Errors:

Type
Line
Description

error
7
Name of argument $numbers does not match with the DocBlock's name $numbers,... in \sum()

error
7
The type hint of the argument is incorrect for the type definition of the @param tag with argument $numbers in \sum()

notice
7
Parameter $numbers,... could not be found in \sum()

And I can't find anything about it in the v2 docs. Is this still supported in v2? And if yes, how are we supposed to document it?

It should be supported by v3.0.0-alpha. You can just document it as ...$numbers
I think the latter should also work in v2.


Cloud API Keys working with Akash Network.

Currently, api keys cannot communicate with Akash hosted docker containers. The issue is unknown though likely related to cors.
This is an essential feature because it is the cheapest way to host instances and it caps the expenditures in case the keys are leaked.
Along with this completion, we need to add basic instructions for users who'd like to host on Akash.

I did some testing with a custom instance on akash, I wasn't able to reproduce auth/cors related issues, so will be closing this for now.
The instructions will be added with the UI soon as well
This is the build script thats working for me

---
version: "2.0"
services:
  meilisearch:
    image: getmeili/meilisearch
    expose:
      - port: 7700
        as: 80
        to:
          - global: true
    env:
      - MEILI_MASTER_KEY=MEILI_KEY
profiles:
  compute:
    meilisearch:
      resources:
        cpu:
          units: 1
        memory:
          size: 1Gi
        storage:
          - size: 5Gi
  placement:
    dcloud:
      pricing:
        meilisearch:
          denom: uakt
          amount: 10000
deployment:
  meilisearch:
    dcloud:
      profile: meilisearch
      count: 1

I did some testing with a custom instance on akash, I wasn't able to reproduce auth/cors related issues, so will be closing this for now.
The instructions will be added with the UI soon as well
This is the build script thats working for me

---
version: "2.0"
services:
  meilisearch:
    image: getmeili/meilisearch
    expose:
      - port: 7700
        as: 80
        to:
          - global: true
    env:
      - MEILI_MASTER_KEY=MEILI_KEY
profiles:
  compute:
    meilisearch:
      resources:
        cpu:
          units: 1
        memory:
          size: 1Gi
        storage:
          - size: 5Gi
  placement:
    dcloud:
      pricing:
        meilisearch:
          denom: uakt
          amount: 10000
deployment:
  meilisearch:
    dcloud:
      profile: meilisearch
      count: 1


Stopping threshold bug?

The paper says we stop when change in assignment is below the stopping threshold, but the code implements this as:
            if change_in_assign > stopping_threshold:
                flag += 1
            if flag == 4:
                break

This is a bug right? It should be if change_in_assign < stopping_threshold:
I'll fix this in my pull request if so.

Making the fix led to training for much more epochs with improvements on AMI and ACC. If the results in your paper was submitted with the same code with this bug in it, you might get better results with the fix!

The paper says we stop when change in assignment is below the stopping threshold, but the code implements this as:
            if change_in_assign > stopping_threshold:
                flag += 1
            if flag == 4:
                break

This is a bug right? It should be if change_in_assign < stopping_threshold:
I'll fix this in my pull request if so.

No. This isn't a bug. This was put in place to let DCC run until a point it breaks down the clustering. That is why flag is updated couple of times before algorithm is halted. The original criterion is little conservative.
I believe it was added after the paper was done. Overall, we found just minor change in AMI / ACC outputs.


Hide containers filters in default filters if containers UI tab is hidden

@miq-bot add_label providers/containers
@miq-bot add_label ui
@miq-bot add_label WIP
@miq-bot add_label bug
@dclarizio @h-kataria please review, thanks

@dclarizio , @h-kataria , currently this fails with "undefined local variable or method `get_vmdb_config' for ConfigurationController:Class" in one place, although get_vmdb_config is used in the ConfigurationController class prior to this patch too, so it looks like a scoping issue. suggestions how to address it?

@dclarizio , @h-kataria , revised, please review

looks good, verified in UI. @abonas can you remove WIP label

@miq-bot remove_label wip

looks good, verified in UI. @abonas can you remove WIP label

done

@h-kataria wip label removed


How to contribute?

should I make edits on gh-pages or main? I'm confused

ok, i figured it out. The stuff at root level in main is irrelevant and should be deleted? Everything is under /docs and gh-pages is never edited


polyfill node imports

had this working locally but not on gha for some reason...

closing due to fix in another pr


Objects sticking to boundries

Hello, im building a website for a client, and the objects that i had developed happen to stick to the walls or borders around the window screen. When my objects start bouncing around they end up getting stuck on the wall and don't repel back. Any idea how to fix it. The website can be viewed at http://berberianmysterytheatre.com/
Thanks again for the help.

Please have a look here.
https://github.com/liabru/matter-js/issues/67

Thanks @GurpreetSran, also note that upgrading to the latest master build has this set by default.


[CONFIGURATION]:Frontend Configuration work

Please review this PR @unnati914 and @rohan-kulkarni-25 . I have added the files for React in this branch
CC: @devkapilbansal

@unnati914 and @rohan-kulkarni-25  please review this as well, These are just the configuration files and really aren't big


Error: Compiling for Electron(Atom Shell)

I am trying to recompile for use with Electron but am running into this error:
><EMAIL_ADDRESS>install /Users/zpickett/Projects/tracker/node_modules/serialport
> node-pre-gyp install --fallback-to-build

Unsupported target version: 0.25.2

node-pre-gyp ERR! install error
node-pre-gyp ERR! stack Error: Unsupported target version: 0.25.2
node-pre-gyp ERR! stack     at get_runtime_abi (/Users/zpickett/Projects/tracker/node_modules/serialport/node_modules/node-pre-gyp/lib/util/versioning.js:140:23)
node-pre-gyp ERR! stack     at Object.module.exports.evaluate (/Users/zpickett/Projects/tracker/node_modules/serialport/node_modules/node-pre-gyp/lib/util/versioning.js:249:19)
node-pre-gyp ERR! stack     at install (/Users/zpickett/Projects/tracker/node_modules/serialport/node_modules/node-pre-gyp/lib/install.js:138:31)
node-pre-gyp ERR! stack     at Object.self.commands.(anonymous function) [as install] (/Users/zpickett/Projects/tracker/node_modules/serialport/node_modules/node-pre-gyp/lib/node-pre-gyp.js:48:37)
node-pre-gyp ERR! stack     at run (/Users/zpickett/Projects/tracker/node_modules/serialport/node_modules/node-pre-gyp/bin/node-pre-gyp:79:30)
node-pre-gyp ERR! stack     at Object.<anonymous> (/Users/zpickett/Projects/tracker/node_modules/serialport/node_modules/node-pre-gyp/bin/node-pre-gyp:131:1)
node-pre-gyp ERR! stack     at Module._compile (module.js:431:26)
node-pre-gyp ERR! stack     at Object.Module._extensions..js (module.js:449:10)
node-pre-gyp ERR! stack     at Module.load (module.js:356:32)
node-pre-gyp ERR! stack     at Function.Module._load (module.js:311:12)
node-pre-gyp ERR! System Darwin 14.3.0
node-pre-gyp ERR! command "/usr/local/bin/iojs" "/Users/zpickett/Projects/tracker/node_modules/serialport/node_modules/.bin/node-pre-gyp" "install" "--fallback-to-build"
node-pre-gyp ERR! cwd /Users/zpickett/Projects/tracker/node_modules/serialport
node-pre-gyp ERR! node -v v2.0.1
node-pre-gyp ERR! node-pre-gyp -v v0.6.7
node-pre-gyp ERR! not ok

Version 0.25.2 is the electron version. I also seem unable to find/understand where the version is being locked/limited?

I've also tried everything I could find (including https://gist.github.com/jedthehumanoid/a7f8278e0a37d259adca) but in vain.

https://github.com/voodootikigod/node-serialport/issues/538#issuecomment-115207973

If anybody stumbles upon this while trying to use node-serialport with electron: there is a temporary solution in the form of the [serialport-electron](https://www.npmjs.com/package/serialport-electron).
Hopefully, in the long run, node-pre-gyp will support electron OOTB.


refactor: change some hashbrown RawTable uses to HashTable (round 3)

Which issue does this PR close?
For #13433, but only parts of it.
Note this that PR is rather small because it's -- in contrast to the other outstanding migrations -- rather self-contained.
Rationale for this change
Prepare hashbrown 0.15 upgrade.
What changes are included in this PR?

a few migrations

Are these changes tested?
Existing tests pass.
Are there any user-facing changes?
No.

Thank you @crepererum  and @findepi


Navisworks Connector: Resolve problem can't load assembly in first time click button

Description & motivation

Changes:

SDK Microsoft.NET.Sdk.WindowsDesktop Can change to Microsoft.NET.Sdk
Resolve Assembly before load ribbon.

To-do before merge:

Screenshots:
Before :

After :

Validation of changes:

Checklist:

[ ] My pull request follows the guidelines in the Contributing guide?
[ ] My pull request does not duplicate any other open Pull Requests for the same update/change?
[ ] My commits are related to the pull request and do not amend unrelated code or documentation.
[x] My code follows a similar style to existing code.
[ ] I have added appropriate tests.
[ ] I have updated or added relevant documentation.

References
@jsdbroughton

What is intended to resolve? Is this "correcting" a problem with functionality or with messages while debugging the project?

Hi @jsdbroughton, this pull request resolve problem from my side. With the first time when you click to button Speckle Connector Navisworks , Navisworks will be exits.

Hi @jsdbroughton, after Navisworks startup, ribbon with name Speckle show success in this process, but when I click button to open panel Speckle Connector, then it Crash Navisworks.

Thanks I'll review.
Either linking to an issue(s) or fully describing what the PR is for is helpful context.

Thank you. I modify some text to clearly problem.

Hi @jsdbroughton , I think I need some day to explore all process workflow of code first before I can create pull request fix anything clearly, I saw may problem need to test more time than I think.


Detach netvm fails for mirage v0.8.2

Attempting to detach mirage-firewall (v0.8.2) as netvm fails, and leaves the vm without a network connection - unable to change the netvm.

qvm-prefs VM netvm mirage-firewall
qvm-prefs VM netvm x
error: no such property 'netvm'
qvm-prefs VM
netvm: mirage-firewall

journalctl shows issue with netvm detach.
VM has no network connection
Other VMs can still be attached to mirage-firewall, so its just an issue with detaching from mirage.
Submitting this issue here as only encountered this with mirage.

I just pushed a PR on mirage-net-xen that allows to dynamically change the netvm for any client VM without qvm-prefs: error: no such property: 'netvm' anymore.

Thanks for your report, this has been fixed in the 0.8.3 release. Please re-open if it still fails for you.


DL-140 : Fix distributedlog-core findbug inconsistent synchronization warings

@xieliang could you verify if the failure is due to your change ? Is the test passing on your local environment ?
cc @sijie

@franckcuny I passed the travis ci for both scala 2.10 and 2.11. it should be transient errors.

@xieliang the change looks good. but I am interested to see how did you find this. I thought we already enable findbugs on ci jobs. I am wondering why didn't we detect the findbug warnings.

I ran the findbug (3.0.1) binary on my local devbox. Those multi-thread warings exist even after importing the findbug exclude xml file manually,  what's the findbugs binary version at ci side, i guess it's obsolete...

@xieliang I think currently we are using 3.0.3 in the pom.xml file.
          <plugin>         <groupId>org.codehaus.mojo</groupId>         <artifactId>findbugs-maven-plugin</artifactId>         <version>3.0.3</version>       </plugin>
The travis ci runs "findbugs:check" for every pull request - "mvn --batch-mode clean package findbugs:check"
is it the same command that you are running?

It's not the same thing. The travis ci runs with "findbugs-maven-plugin", and i ran with findbugs binary package.
(From http://gleclaire.github.io/findbugs-maven-plugin/index.html , i see the core findbugs version are all 3.0.1.)
Let me finger out the diff later

@sijie , in distributedlog-core pom.xml, if you add "Low" after ""  config item, you will see sth like this:
[INFO] BugInstance size is 120
[INFO] Error size is 0
[INFO] Total bugs: 120
Just like my result from findbugs gui view.

@xieliang gotcha. it might be worth having a jira to make the findbugs threshold 'LOW'. and in this jira, we need to fix those findbug warnings.
for this pr, I think it is good to fix synchronization issue.

@xieliang do you mind rebasing this pr to latest master? so it would trigger the checks again.

Rebased.
"it might be worth having a jira to make the findbugs threshold 'LOW'. and in this jira, we need to fix those findbug warnings."  , i'll create a new jira.


#19605 was closed and not resolved

See #19605

Feel free to open up an issue, using the issue template and with proper data.
This title & the contents of this issue, is not helping anybody.
Thanks! 👍


[Task] Refactor Generate page form

Goal is to make

the least decision on frontend side,
code more readable,
extensible and less error-prone.

Tasks to do

[x] Remove Plugin package name mapping constant from frontend
[ ] Have custom component name sent from backend for a plugin
[ ] Make GeneratePageForm as a base component that is extended for different plugins

Need to remove this from frontend

export const PLUGIN_PACKAGE_NAME = {
  MONGO_DB: "mongo-plugin",
  S3: "amazons3-plugin",
  GOOGLE_SHEETS: "google-sheets-plugin",
  FIRESTORE: "firestore-plugin",
};

export const PluginFormInputFieldMap: Record<
  string,
  { DATASOURCE: string; TABLE: string; COLUMN: string }
> = {
  [PLUGIN_PACKAGE_NAME.MONGO_DB]: {
    DATASOURCE: "MongoDB",
    TABLE: "collection",
    COLUMN: "field",
    FORM_UI: "default" // this can be empty also
  },
  [PLUGIN_PACKAGE_NAME.S3]: {
    DATASOURCE: "S3",
    TABLE: "bucket",
    COLUMN: "keys",
     FORM_UI: "s3_form"
  },
  [PLUGIN_PACKAGE_NAME.GOOGLE_SHEETS]: {
    DATASOURCE: "Google Sheets",
    TABLE: "spreadsheet",
    COLUMN: "keys",
    FORM_UI: "google_sheet_form"
  },
  DEFAULT: {
    DATASOURCE: "SQL Based",
    TABLE: "table",
    COLUMN: "column",
    FORM_UI: "default" // this can be empty also
  },
};

@abhvsn you can refer to this for backend data that needs to be sent with the plugin.
I added the FORM_UI field which has the name of the form the frontend needs to render.

@Rishabh-Rathod can we close this if already been taken care of?

@Rishabh-Rathod can we close this if already been taken care of?

The frontend part of it is not yet completed.


Separate network protocol from the rollback logic

I don't know how tightly coupled they are but it would be nice if they could be isolated from each other and maybe show how to combine them in an example. For example if I use an engine which has its own protocol I might want to skip ggpo's implementation and only use the rollback.

This pull request allows the user to inject their own network object, though there are issues with C support and it will be in limbo for a while. https://github.com/pond3r/ggpo/pull/42


Address filtering is unimplemented

Any intentions to finish the implementation soon?

Actually, I haven't decided yet.
I thought it was a good idea at the very beginning of the project. The server (receiver) side would also allow '*' and '?' symbols to filter incoming messages.
Then, I read the OSC spec again, and you the routing and pattern matching part of it, and I was puzzled: the server side should not contain any '*', but OSC messages could. So it is basically the reverse of my initial intent...
Support both pattern matching would be a mess, so I must choose between:

implement the spec, even if I find it awkward and I doubt it is used this way by anyone, or
be practicle, trash this part of the spec, be haunted by Matt Wright for ever.

What are your expectation for this feature ?

Hélo, was about to ask the same question.
Say you receive several OSC bundles at the same time on different addresses, is there at this point a way to distinct them ?
I never managed to do so, but I'm clearly not a blueprint roxxxxor so I may have missed something.
That's what I'd expect from this feature.

Another "better late than never" message: I chose to implement the simpliest version of it, do not receive messages that do not start with the filter.
No *, ?, etc.
It is mainly intended as an optimization. (pre-filter in C++ is faster)
You may use a blueprint "switch" in addition to route the message.


[FR Jetcaster]: Rethink empty, loading, success, and error states in

Is there an existing issue for this?

[X] I have searched the existing issues

Is this a feature request for one of the samples?

[X] Yes, this is a specific request related to this samples repo.

Sample app
Jetcaster
Describe the problem
Jetcaster uses a sealed interface to display different states of the UI. For example:
sealed interface ScreenUiState {
    data object Loading : ScreenUiState

    data class Error(
        val errorMessage: String? = null
    ) : ScreenUiState

    data class Ready(
        /* Params here */
    ) : ScreenUiState
}

However, this may not be the best set up UX-wise as intermediate error states would result in the entire screen showing an error vs. showing an error as part of a partially rendered screen.
Describe the solution
A possible solution would be to fold loading and error states into a single screen state data class.
data class ScreenUiState(
    isLoading: Boolean,
    errorMessage: String?,
    /* Params here */
)

Other proposals are welcome.
Additional context
See discussion in https://github.com/android/compose-samples/pull/1363#discussion_r1581044482
Code of Conduct

[X] I agree to follow this project's Code of Conduct

I personally think that managing multiple distinct states within a single object can make Compose code less readable and harder to manage.
Hmm... If I think about it simply...
I believe that managing each state separately could allow for more versatile responses in various situations.
private val refreshing = MutableStateFlow(false)
private val _state = MutableStateFlow<ScreenUiState>()
private val _error = MutableStateFlow("")

combine (
    // ...
)
    .onStart { refreshing.value = true }
    .onEach { _state.value = it }
    .catch { throwable -> _error.value = throwable.message }
    .collect { refreshing.value = false }

what do you think..?

I personally think that managing multiple distinct states within a single object can make Compose code less readable and harder to manage.

Yes, that was the impetus for using the sealed interface approach though that did create some limitations as per the description of this issue.
I could imagine a scenario where you might have multiple loading/error states on the same screen. In that case, I would prefer to use a Result in the screen state instead of having a separate flow for loading / errors. For Jetcaster, however, the solution you proposed could work since there aren't any screens where that scenario could apply.

I could imagine a scenario where you might have multiple loading/error states on the same screen. In that case, I would prefer to use a Result in the screen state instead of having a separate flow for loading / errors.

Yes, I also think that using Result or sealed types is more efficient in typical scenarios
After reading the comment by mlykotom, I imagined a scenario where the Success UI is maintained while the Loading UI appears simultaneously. Based on this, I suggested this because I think it might be more efficient to manage the states separately. 🤔
Thank you.


ggcyto vs FlowJo axis transformation

Hi, I am wondering why the x-axis (CD154) looks like this in FlowJo:

but like this in ggcyto: (the low end of the CD154 population ends more "bluntly")

My guess is that the same events are being plotted in both FlowJo and ggcyto, but the axis transformation used for visualization is a little different, which makes it look more blunt in ggcyto. That would be the best scenario, as opposed to the alternative, in which I'm losing some CD154- events somewhere along the way.
Just wondering if this is a known issue that someone has insight into.
Thank you!

Could this share this sample and workspace so we can troubleshoot?

Just sent to you and Jake. Thanks!

Thanks. I'll take a look in a little bit.

Hi @malisas , sorry I'm getting to this a little late. I actually cannot open that workspace from FlowJo 10.6.1. I just get an "Error Adding Files" notification. I can, however, open it and create as a GatingSet using CytoML as in your Rmd. I'm still investigating that issue.
I do recall there being an issue with one of the transformations whose definition changed between FlowJo versions that may be the cause of this depending on whether we are handling both sides of the change in CytoML. I'm looking back through my records now to figure that out.
But, on the plus side, I can confirm that no cells are being lost:
> gh_pop_get_data(gs[[1]])
cytoframe object '15530_H12_H12_096.fcs'
with 215775 cells and 21 observables:

Anyway, I'll update here when I know more.

I think the issue of opening it is because the workspace is still explicitly pointing to the original home of the files. I can try to fix that in the xml, though.

@malisas , so the compensation matrix looks a little suspect, but I think the blunting you're seeing is just a result of your inclusion of marginalFilter for filtering the plotted data:
ggcyto(gs[[1]],
       aes(!!currentXaxis, !!currentYaxis),
       subset = parentGate,
       filter = marginalFilter) +
    geom_hex(bins=128) +
    geom_gate(myGate) +
    axis_x_inverse_trans() + axis_y_inverse_trans() +
    coord_cartesian(xlim = c(-100, 4000), ylim = c(-100, 4000))

Removing the marginal filter and adjusting the bins accordingly:
ggcyto(gs[[1]],
       aes(!!currentXaxis, !!currentYaxis),
       subset = parentGate) +
    geom_hex(bins=1024) +
    geom_gate(myGate) +
    axis_x_inverse_trans() + axis_y_inverse_trans() +
    coord_cartesian(xlim = c(-100, 4000), ylim = c(-100, 4000))

Actually, one better, if you avoid the y-dimension gate extension logic by passing a large negative value to extend_val in flowjo_to_gatingset (see Greg's discussion here), you will get even closer to what you have in FlowJo :
gs <- flowjo_to_gatingset(ws, name=sampleGroup, keywords=keywords2import,
                             path=fcs_subfolder, extend_val = -10000)

...
ggcyto(gs[[1]],
       aes(!!currentXaxis, !!currentYaxis),
       subset = parentGate) +
    geom_hex(bins=128) +
    geom_gate(myGate) +
    axis_x_inverse_trans() + axis_y_inverse_trans() +
    coord_cartesian(xlim = c(-100, 4000), ylim = c(-100, 4000)) +
    geom_stats()

Hi Jake, thank you so much for troubleshooting this.
I had gotten into the habit of using marginalFilter every time I plotted something with ggcyto (to make the hex bins look nice), but I will use it more judiciously from now on to avoid things like this.
About the gate extension thing, it seems that:

Setting extend_val = -10000 will result in similar gate membership in openCyto and FlowJo (as seen by the geom_stats() number), but
(according to the linked issue) this setting may actually be undesirable because openCyto is correcting a FlowJo export issue.
So perhaps I should try out both ways to make sure that the biological and statistical conclusions will not be too affected by this setting.

Yep, in general the extend_val logic is still necessary at least up through FlowJo 10.6.1, although not for ellipsoidGates. More discussion here. So you're right that you may need to try it both ways, particularly for older FlowJo versions.

Just an update about extend_val:
I'm very glad that you mentioned the extend_val issue to me, because I experienced a problem related to it today.
With the default extend_val=0 setting, ggcyto was displaying an artifact in my dataset that was not showing up in FlowJo (see left side of figure below).
It turned out that the L/D gate was getting extended in flowjo_to_gatingset to include those events (which also show up as an odd line when looking at CCR7 vs CD45RA):

So I guess I should be using extend_val=-10000 (or some large negative number) for this dataset at least.
Thanks again for your insight!


How to stop auto column name uppercase

Hi to everybody,
yesterday i tried to add a new column to a table using DBeaver using a camel case name, but every time i saved it DBeaver automatically changed the name to upper case. I have already added the column to my table using a script, but i am still curious to know if it is possible to turn off this feature on DBeaver.
Could somebody help me to solve this problem? I found some informations on internet but nothing worked.
Best regards.

Hello @AleramoAl
This is the default behavior, yes.
You can change it by turning on this setting:

Thank you very much! Yesterday i spent a long time to search for an answer.


What hidden states should I use?

the output of encoder is N*d states for each text input;

simcse use the 1st one as the output.
the sentence-transformers use the mean pooling(only for attention mask=1 states).

Your default is the same as mean pooling?

yes, use the mean pooling(only for attention mask=1 states)


Using / for API reference

Hi there,
I'm using '/' as route on the config so that the main page is the API Reference (the Redoc component). However, no matter if I use the cusom page or custom layout, I get stuck into "Loading".
When using a different route like '/api', it works fine.
Any clue? Thx!

Are using both the route option and also creating a custom page using index.jsx/tsx file in the pages directory? Then these options might conflict, you can remove one of them and I think it should work.
If you have a link to the repo that would be great to debug this

With route is completely removed from the config, and using https://github.com/rohit-gohri/redocusaurus/blob/main/website/src/pages/examples/custom-layout/index.jsx as my only src/pages/index , it's stuck on Loading.
However, now I tried to remove all the /src and using route: '/' on config and it loads fine.
Could it be that import useSpecData from '@theme/useSpecData'; does not load at all?

@rohit-gohri here you have an example config and page https://github.com/jrub/redocusaurus_test

Figured it out. The id in useSpecData is supposed to be the same id you specify in the docusaurus config (here https://github.com/jrub/redocusaurus_test/blob/63a8e30710d18c586e835ff3d41047f450091a2b/docusaurus.config.js#L66). Let me know if this is not the fix.
I guess it's not very clear in the docs. Added it as a TODO to the existing docs ticket (https://github.com/rohit-gohri/redocusaurus/issues/147).

Cool! That worked, thank you 🙏


Feature Toggles

Allow toggling features using hotkeys:

~Ctrl+Shift+N: Toggle Names distribution. This will enable/disable generated names.~ Redundant as the mod can now be safely removed anytime.
RCtrl+O: Toggle Obscuring distribution. This will enable/disable obscured names.
RCtrl+L: Reload settings.
RCtrl+RShift+G: Regenerate all names. Show a message box with confirmation.
RCtrl+G : When crosshair points to an NPC, regenerate this NPC's name. No confirmation.

Probably allow customizing these hotkeys in INI.

Need to add Saving options into ini file when toggling obscurity.