[Fix #12761] Fix a false positive for Style/HashEachMethods

Fixes #12761.
This PR fixes a false positive for Style/HashEachMethods when the key block argument of Enumerable#each method is unused after chunk.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Looking good!


[Broken] Pornhub download ouput title not including chinese

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 reporting that yt-dlp is broken on a supported site
[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 checked that all URLs and arguments with special characters are properly quoted or escaped
[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
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
America
Provide a description that is worded well enough to be understood
I'll go https://cn.pornhub.com/ When opening the video details page, you can see that the video title contains a mixture of Chinese and English, but when downloading through yt dlp, there are no Chinese characters, and using% (title) s is also different from the title on the website.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] 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
yt-dlp -vU --proxy socks5://<IP_ADDRESS>:1081/ -i "https://cn.pornhub.com/view_video.php?viewkey=66206be6c5552"
[debug] Command-line config: ['-vU', '--proxy', 'socks5://<IP_ADDRESS>:1081/', '-i', 'https://cn.pornhub.com/view_video.php?viewkey=66206be6c5552']
[debug] Encodings: locale cp936, fs utf-8, pref cp936, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [ff0779267] (win_exe)
[debug] Python 3.8.10 (CPython AMD64 64bit) - Windows-10-10.0.22631-SP0 (OpenSSL 1.1.1k  25 Mar 2021)
[debug] exe versions: ffmpeg 7.0-full_build-www.gyan.dev (setts), ffprobe 7.0-full_build-www.gyan.dev
[debug] Optional libraries: Cryptodome-3.20.0, brotli-1.1.0, certifi-2024.02.02, curl_cffi-0.5.10, mutagen-1.47.0, requests-2.31.0, sqlite3-3.35.5, urllib3-2.2.1, websockets-12.0
[debug] Proxy map: {'all': 'socks5://<IP_ADDRESS>:1081/'}
[debug] Request Handlers: urllib, requests, websockets, curl_cffi
[debug] Loaded 1810 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)
[PornHub] Extracting URL: https://cn.pornhub.com/view_video.php?viewkey=66206be6c5552
[PornHub] 66206be6c5552: Downloading pc webpage
[PornHub] 66206be6c5552: Downloading m3u8 information
[PornHub] 66206be6c5552: Downloading m3u8 information
[PornHub] 66206be6c5552: Downloading m3u8 information
[PornHub] 66206be6c5552: Downloading m3u8 information
[PornHub] 66206be6c5552: Downloading m3u8 information
[PornHub] 66206be6c5552: Downloading JSON metadata
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), channels, acodec, size, br, asr, proto, vext, aext, hasaud, source, id
[debug] Default format spec: bestvideo*+bestaudio/best
[info] 66206be6c5552: Downloading 1 format(s): hls-3309-1
[debug] Invoking hlsnative downloader on "https://cv-h.phncdn.com/hls/videos/202404/18/451223041/,1080P_4000K,720P_4000K,480P_2000K,240P_1000K,_451223041.mp4.urlset/index-f1-v1-a1.m3u8?kw92RaktPaIlpWVxS9oxT71VCmzk7Aqya8Ig7S5c6AmIh3eE_Nih8ip5qxaQN7gHCmisdGKWYzPK9OR9ss7W43TJGSyjZaMkryvqc3bpDrqfXT-x59WOIX6mGQ8WZUyNaedr6kZ80OJbTRB8tuyUD3Zp8X5Rr7qPj9ZRU0ZRFIvVBH4L_dX2ctOb_T72JTk1TC-OBewf9w"
[hlsnative] Downloading m3u8 manifest
[hlsnative] Total fragments: 333
[download] Destination: Slutty female tutor [66206be6c5552].mp4
[debug] File locking is not supported. Proceeding without locking
[download]   1.9% of ~ 376.19MiB at  823.27KiB/s ETA 10:26 (frag 6/333)

This is because the extractor fetches the website from www.pornhub.com not cn.pornhub.com. And, I agree that it's weird not to at least communicate this. It even prints out:
[PornHub] Extracting URL: https://cn.pornhub.com/view_video.php?viewkey=66206be6c5552
and then the extractor downloads https://www.pornhub.com/view_video.php?viewkey=66206be6c5552 which has no Chinese in the title.


[Feature Request] Per Subdomain Secret

Please add the ability to specify a dedicated secret for a given subdomain.
E.g. via environment variable
docker run -it -d \
    -p 8080:8080 \
    -p 53:53 \
    -p 53:53/udp \
    -e SHARED_SECRET=changeme \
    -e SECRET_mysubdomain1=changeme1 \
    -e SECRET_mysubdomain2=changeme2 \
    -e ZONE=example.org \
    -e RECORD_TTL=3600 \
    --name=dyndns \
    davd/docker-ddns:latest

I created a pull request #59 where it is possible to use ddnskeys to have restricted access to a subdomain as defined in the zone file with update-policy. The shared secret will be still the same for all domains but further checking is done by the bind9 server with the ddnskey. Therefor you have to make sure that the default ist to not have access without a ddnykey.
This method may break compatibility with existing systems since it adds a variable to the request. For the future it may be a good idea to remove the current "shared secret check" and replace it with ddnskeys in a way to keep compatibility. By using ddnskeys we can archive ACL by delegating the task to the bind9 server itself.
But there is not much support for it right now and to get it to work requires some manual adjustments.


Show a Popup in Browser (IDFGH-11801)

Is your feature request related to a problem?
there is a Function:
httpd_resp_send_err
which can show such a Message in Browser e.g. Chrome:

Describe the solution you'd like.
I would like a httpd_resp_send_msg which does the same as httpd_resp_send_err but not show "500 Error!" but e.g. "Success" ...
Describe alternatives you've considered.
No response
Additional context.
No response

would be interesting Solution for https://github.com/espressif/esp-idf/issues/11220

Hello @diplfranzhoepfinger
I think this might help you: https://github.com/espressif/esp-idf/blob/master/components/esp_http_server/include/esp_http_server.h#L1318
Please let me know if it works.
Thanks!

i saw in the File Server you do it this way:
https://github.com/espressif/esp-idf/blob/b3f7e2c8a4d354df8ef8558ea7caddc07283a57b/examples/protocols/http_server/file_serving/main/upload_script.html#L49


Guard distance

Its me again :)
I want some NPCs to guard me. But they dont find the way through doors etc. So the question is, if I can change the distance of the guardian to follow me. With a short distance he would find the way on corners or doors. If there isn´t this option, can u add it?
Greetings, Firemonkey

Definitely could add an option for that!

Yeah. You are my man 👍

Sorry for the weirdly long (one and a half years!) delay on this! I've been distracted and forgot I had left some not too difficult to add issues open.
I've added new config options for this (3 of them, under 'guard follow distance').
The 3 of them are separate due to the feature added following issue #258 (see there for some relevant notes)
This fix is included in Sentinel build DEV-183+ http://ci.citizensnpcs.co/job/Sentinel/


Add Queue typing

What kind of change does this PR introduce?
refactoring, typing
Did you add tests for your changes?
no
If relevant, link to documentation update:
n/a
Summary
Add complete typing for Queue.
Does this PR introduce a breaking change?
no
Other information
There is an issue with generics.

Support for @template on classes will be in typescript@next starting tonight. That should fix the build errors with "Cannot find name 'T'."

Thanks


Add serviceaccounts to prioritized resources

Add serviceaccounts to the default list of prioritized resources used
when restoring.
Signed-off-by: Andy Goldstein<EMAIL_ADDRESS>Fixes #239

LGTM.


[Bug]: input field validation doesnt work

Before submitting...

[X] I have searched for duplicate or closed issues.
[X] I have read the CONTRIBUTING document and my issue is following the guidelines.
[X] I have read the template completely before filling it in.

Context
IIn Materialize 1.0.0, input field validation occurred automatically by adding the attributes data-error and data-success. Additionally, enabling regex validation for input fields would enhance functionality.
Current Behavior
The data-error does nothing when you write an invalid e-mail for example.
Codepen: https://codepen.io/lschwend98/pen/wvNXvbz
Expected behavior
Validate the type and add the error or success mesage to the input field.
Possible Solutions or Causes
Maybe the functionality was removed and could be added again?
Steps to reproduce
No response
Your Environment

Version used: 2.0.3 beta
Browser Name and version: Google Chrome Version 119.0.6045.160
Operating System and version (desktop or mobile): desktop
Additional information you want to tell us:

Hi, yeah thanks for reporting it. I have created a PR for this fix. Apparently, many functions were removed due to jQuery. I'm sure this might affecting other components as well. While you're here, please do test it out and check other types of input as well. Thanks :)

Thank you for your PR, looks good. How can I test, if it also works with type="date" or type="number" for Example? Also would be great if it works with required attribute?

I dropped myself from fixing this issue. Sorry, I don't currently have time to complete this.

@Jerit3787 could you provide me a brief description on what still needs to be done to address this issue?

*Update: As I looked a bit deeper into this issue, as there is HTML5 validation introduced in 2013, this seem to work properly on submit in combination with the Materialize library, the above CodePen example is apparently missing the submit button. In case of form input without a submit button, with AJAX for example, check out the Javascript validation API:

https://stackoverflow.com/questions/11866910/how-to-force-an-html-form-to-validate-without-submitting-it-via-jquery
https://www.w3schools.com/js/js_validation_api.asp

CodePen example

@Jerit3787 could you provide me a brief description on what still needs to be done to address this issue?

Yeah, so far there materialize doesn't check for any validation methods, namely email and others. In my PR, I've reimplemented the validation feature that was removed before this. You can take a look at it for any reference

*Update: As I looked a bit deeper into this issue, as there is HTML5 validation introduced in 2013, this seem to work properly on submit in combination with the Materialize library, the above CodePen example is apparently missing the submit button. In case of form input without a submit button, with AJAX for example, check out the Javascript validation API:

https://stackoverflow.com/questions/11866910/how-to-force-an-html-form-to-validate-without-submitting-it-via-jquery
https://www.w3schools.com/js/js_validation_api.asp

CodePen example

Yeah the HTML5 validation still works, but there is no feedback in terms of the Text Input component like a red bar showing error. In previous implementation, materialize will check once user leaves focus (i think?) or press enter on keyboard and instantly shows a red if the validation found an error on the user inputs.

I've reviewed PR #438, which works as expected, except that the styling for handling invalidity of the field was still missing
Regarding my previous notation on HTML5 validation: there is a possibility to extend existing Materialize styling to the invalid CSS pseudo selector, however implementing an email type input which has attribute required makes it a bit more complex as expected, as mentioned in this comment by Jerit3787. The field is marked as invalid immediately since there is no value, not sure if there is any workaround for this, need to dig a bit deeper into the HTML5 validation on a required (empty) field
A new PR (#521) with enhancements and additions has been created
@Jerit3787 I've refactored the code slightly and removed valid class from a point of view where input should always be considered valid as default except if a invalid error status is thrown
Missing CSS classes and styling for handling the invalidity of the field have been added


fixed environment load, added pbtxt export

Ensure that all of the necessary environmental dependencies are checked for and installed. (added flask, absl)
Added reactions datatable export to the end of the notebook (similar to example 1)
Validated reactions datatable after export

Hi @ipendlet, anything we can do to help with this?


Add facades for PrimeReact's ToggleButton, Message, ProgressSpinner

Also very minor changes and our own facade for TooltipOptions.

Do you want to use prime's tooltips instead of floating ui?

Do you want to use prime's tooltips instead of floating ui?

I couldn't figure out how to (more or less easily) attach a floating ui tooltip to a prime component, since the facade seems to need a tag to attach to, not a component.

Do you want to use prime's tooltips instead of floating ui?

I couldn't figure out how to (more or less easily) attach a floating ui tooltip to a prime component, since the facade seems to need a tag to attach to, not a component.

There is a syntax "withTooltip" you can use on components to add one

There is a syntax "withTooltip" you can use on components to add one

I'll give it a try, thank you.

There is a syntax "withTooltip" you can use on components to add one

It only seems to be available for VdomTag, forcing us to enclose all components in an extra tag. As discussed on Slack, maybe we should use Prime's mechanism for Prime components? I'll leave this in the facade for now, we can refactor later if we change our decision.


About decoder

I would like to ask about how the decoder is implemented, is there any detailed code? Thank you !

You can see the detailed code of the decoder at Line 50 in https://github.com/daiquocnguyen/R-MeN/blob/master/RMeN_tf/model_R_MeN_TripleCls_CNN.py
or Line 83 in https://github.com/daiquocnguyen/R-MeN/blob/master/RMeN_pytorch/RMeN.py


Local Network Support?

Hello! Any chance you can add support for local network folders (Windows 10)? I have my projects stored locally on my laptop but also need to push modified files on Save to a local network folder without using FTP/SCP.
Reason being is that we have a dev web server setup where everyone can test/view their changes if they don't have one setup on their local machine.
Thank-you!

Hi, i suggest you to install a little FTP server on that machine or to enable the Microsoft FTP server:
Press WIN+R, type OptionalFeatures.exe  in the window and click OK.
Go to Internet Information Service > FTP Server > and select FTP Service.
You will also need to go to Web Management Tools and select IIS Management Console
Once finished press again  WIN+R, type InetMgr.exe in the window and click OK.
Add one FTP site trough the panel on your left, put your info like site name, the path that you’d like to serve and click Next. At binding and SSL settings, select “No SSL” (you are working in local safe lan, right?) and click Next. Select Anonymous for authentication for simple config, select Anonymous users for authorization and enable the read and write permissions.
Bye!

Attach the local network folder in Windows (and open). Click right click and Open with Atom. Any changes will be made.


Incorrect coloring and syntax error

Applies To

[X] Notebooks (.ipynb files)
[ ] Interactive Window and/or Cell Scripts (.py files with #%% markers)

What happened?
Variable names have several colors at the same time.

VS Code Version
1.68.1
Jupyter Extension Version
v2022.5.1001601848
Jupyter logs
No response
Coding Language and Runtime Version
Python 3.9.12 by Miniconda
Language Extension Version (if applicable)
Python v2022.8.1
Anaconda Version (if applicable)
conda 4.13.0
Running Jupyter locally or remotely?
Local

Thanks for filing. I don't have enough info here to exactly repro this, but this type of coloring mismatch it usually due to a disconnect between the notebook and Pylance, which is supplying the coloring for the syntax here.
Could you please pull the logs from Output->Jupyter? Might help up with understanding this scenario. @rchiodo are there also Pylance logs we should be getting here?

Yes there are settings for pylance. Unfortunately these depend upon the experiment state of the machine before we can get logs for pylance (well because the experiment determines what way pylance is started).

closing this since it has been over a month since the last request for info


FluoConfiguration should derive table name from application name if not set

FluoConfiguration requires the application name to be set. However, it does not require the table name to be set (defaults to "fluo"). Problems can occur if separate applications use the same table. The table name should be derived from the application name, if not explicitly set. Or, the table name should be required.
Additionally, the configuration should have more javadocs explaining what the application name is used for, and how it relates to the table name, and explaining the pitfalls of using the same table for different applications.

If table name was required, the application name could be automatically derived from the table name.

I think this was handled but would like to confirm. I think basically it was made so the table name is required.


Add field for storing short host name for server to be monitored

Today there is a "host" field (for FQDN and port to connect to) and a "serverName" field (for a description of the server).
It would be useful to have a field containing just the host name, without the FQDN etc.
This would make it easier to use Grafana templates, for example.

Fixed by #48


issue-2768: added new option allow_only_external_registration

#2768

Codecov Report

Merging #3910 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #3910      +/-   ##
==========================================
- Coverage   20.16%   20.15%   -0.01%
==========================================
  Files         145      145
  Lines       29171    29172       +1
==========================================
  Hits         5881     5881
- Misses      22395    22396       +1
  Partials      895      895

Impacted Files
Coverage Δ

routers/user/auth.go
0% <0%> (ø)
: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 e6b0a2d...1ad386e. Read the comment docs.

LGTM


Kata Containers 1.13.0-alpha1

config: remove CONFIG_REFCOUNT_FULL in config.base for arm
static-build: Pull the release binary of cloud-hypervisor first
ci: Fixes for virtiofs stable
snap: tag yq version

be5c573 config: bump kernel config
533751a kernel/config: enable efi_rtc support for arm64
51c8ff9 static-build: Pull the release binary of cloud-hypervisor first
0dc7fb4 snap: Fix build for virtiofs
0db1700 obs: Fix build for qemu package
7b6986b ci: obs: Fix default branch in tests repository
9eec02e kernel: ACPI: Always build evged for stable kernel
8def37f qemu: Install virtiofs with vanilla qemu.
6da4b5f config: remove CONFIG_REFCOUNT_FULL in config.base for arm
fe95b20 snap: tag yq version

/test

/test

/retest-obs

We're closing this one as we'll go directly to the 1.13.0-rc0 one.


Fix the black circle with the github icon, which is for some reason ever-so-slightly squished

I've got the circles constrained to 36x36, but I can't get the rendering of Font Awesome's icons to be equally sized/centered within them just yet.

Hrm. I'll try to help out and play around a bit with this as well.


Duplicated Traversy Media Youtube link

Currently the Youtube Channels list contains two links to Traversy Media's channel, I created a pull request ( #297 ) fixing this.

Solved in https://github.com/markodenic/web-development-resources/pull/301


Some i18n configuration options throw warnings in NextJs 12.2.3

🐛 Bug Report
Warning example:
warn  - Invalid next.config.js options detected:
[
  {
    "instancePath": "/i18n",
    "schemaPath": "#/properties/i18n/additionalProperties",
    "keyword": "additionalProperties",
    "params": {
      "additionalProperty": "fallbackLng"
    },
    "message": "must NOT have additional properties"
  },
  ...
]

For me this happens with fallbackLng, keySeparator, nsSeparator, localePath, reloadOnPrerender, but I'm assuming it would happen with each property that is not mentioned in I18NConfig, which is
export interface I18NConfig {
    defaultLocale: string;
    domains?: DomainLocale[];
    localeDetection?: false;
    locales: string[];
}

To Reproduce
Update to nextjs 12.2.3, add one of the mentioned properties to i18n section and run build.
Expected behavior
No warnings.
Your Environment
"next": "^12.2.3",
"next-i18next": "^11.3.0",
...
"i18next@npm:^21.8.13":
  version: 21.8.14

windows

Folks from nextjs said that the issue is on this end

Try:
module.exports = {
	i18n: {
		defaultLocale: "en-GB",
		locales: ["en-GB"],
	}
        keySeparator: "."
};

Yep, I am defining them inside. Outside doesn't warn

Still have the warning in Nextjs v12.2.4
warn  - Invalid next.config.js options detected:
The root value has an unexpected property, returnEmptyString (and fallbackLng) which is not in the list of allowed properties

module.exports = {
	i18n: {
		defaultLocale: "en-GB",
		locales: ["en-GB"]
	},
         fallbackLng: 'en',
         returnEmptyString: false,
};

Where should I put those i18next configs?

@MonstraG Can you confirm?

No, for me everything is ok.
next-i18next.config.js:
const path = require("path");
const languages = require("./i18n.languages.js");

/** @type import("next").I18NConfig */
const i18n = {
	defaultLocale: process.env.DEFAULT_LANGUAGE,
	locales: languages,
	localeDetection: false // disable automatic redirection on the user's preferred locale
};

/** @type import("next-i18next").UserConfig */
const next18nextConfig = {
	i18n,
	fallbackLng: process.env.NEXT_PUBLIC_FALLBACK_LANGUAGE,
	keySeparator: ".",
	nsSeparator: ":",
	localePath: path.resolve("./public/static/locales"),
	reloadOnPrerender: process.env.NODE_ENV === "development"
};

module.exports = next18nextConfig;

next.config.js:
const { i18n } = require("./next-i18next.config");

/** @type import("next").NextConfig */
const config = {
	swcMinify: true,
	optimizeFonts: false,
	compiler: {
		emotion: true
	},
	i18n
};

module.exports = config;

Sorry, it was my mistake that I passed all i18next configs to Next . I missed the instruction https://github.com/i18next/next-i18next#passing-other-config-options that only need to import i18n object for Next config.
Thank you for the reminder ❤


Handle more cases of pyspacer invalid/desynced rowcols

Three commits for pretty much three separate cases. These address most of the job errors and error-notification emails that I've been seeing lately.

Both auto-recovery cases added in this PR got successful examples today.


Can't create a new connection

When i click in the new connection button, i get a blank page, not even the server options are available( image attached).

I am having the same issue.

the only way I was able to make it work was opening the settings.json file and hard coded the connection's config.
if do it, wen you restart the vsc, the extension will connect automatically and you will only have to select the db.
Should looks like the picture (with youre server's info instead).


Tag and update

Would you consider tagging (with something like 2019.12.03) and update the image?

Hi @kenayagi ,
Would be possible. Do you just mean the date at which the Docker image was built? Anything other would be challenging because we use directly git clone the https://github.com/ngoduykhanh/PowerDNS-Admin repository inside our Dockerfile. They do not release specific versions...
https://github.com/HSRNetwork/docker-powerdns-admin/blob/ee545604bfc3305cf36a240812c726c96b6821c8/Dockerfile#L7-L8
Thanks,
Regards,
Philip

Yes. I've seen on upstream that they don't release anything, so I thought that a simple date would at least make the docker image trackable/reproducible.

Is the associated Docker Hub repo already setup for autobuild and tagging?
https://hub.docker.com/r/hsrnetwork/powerdns-admin/tags

Yes it is. It just finished building the image. Docker Hub is super slow at the moment...

Yes it is. It just finished building the image.

Great. Thank you! :thumbsup:


[DRAFT] RFC: or patterns

This is a draft version of an RFC for you to review, before a formal proposal is made for consideration.

Thanks all! Time to :shipit:


Fix the laptop_assignment by returning id

fix retuning in laptoop_assignment
Fix the laptop_assignment by returning id'
Description
Write a summary of your change. Explain why you have made it.
Related to
Make sure you include the issue number with a hash sign # so Github can link this PR to the right issue, like this:
Fixes #1
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have carefully reviewed my own code
[ ] I have commented my code
[ ] I have updated any documentation

(I'm acknowledging also that the package-lock.json is changing here - I think that's just because it's changing back to a format that is generated by the correct, more recent version of npm.)


Update connect-applications-service.md

The targetPort and Port were mixed up, I believe. Are they purposely trying to obfuscate nginx deployment port 80 to 8080 inside the cluster? Why would you want to do that?
If your nginx is 8080 internally and you wish to expose it on 80 inside the cluster, than port: 8080 with targetPort: 80 would seem more correct. However, previous configuration seems to indicate that nginx is running on 80 and 443 internally.

Thanks for this PR. I hope we'll get it reviewed soon.


android 2.3 doesn't work

Investigate and fix

initital fixes
8b58cd9d05ec02397ee49424ce23d47e73dc27cc
00adec6db471262aac76b29fdebfeaaab6031e7f

TODO: fix card spacing and cribbage board UI in 2.3.3


Removed forced padding on CRUD

WHY
BEFORE - What was wrong? What was happening before this PR?
CRUD was forcing a padding and it shouldn't, CRUD should be the most agnostic as possible to any styles.
AFTER - What is happening after this PR?
No more forced paddings.
Is it a breaking change?
Yes, but No 🤷♂
This is for v6.
This was an issue in Core UI, so the fixes for coreui are here;
https://github.com/Laravel-Backpack/theme-coreuiv2/pull/19
https://github.com/Laravel-Backpack/theme-coreuiv4/pull/31

Core UI v2

Core UI v4

Tabler

Excellent PR - thank you @promatik 🙏


BUG: Launcher properties should be set only on Mac

@juanprietob : Can you please look at this pr? Thanks!

The variable SlicerExecutionModel_DEFAULT_CLI_EXECUTABLE_LINK_FLAGS is set by SlicerExecutionModel in the macro 'SEMMacroBuildCLI.cmake'.
If for any reason this is set on other platforms, the command will be ignored.
Instead of checking for 'APPLE', the check should be done for an empty string.

@juanprietob : amended the commit. Please have a look.


Empty response on filter_by_datetime

Any idea about empty response of the  filter_by_datetime function?

incorrect  data when is use filter_by_datetime.
when i used filter_by_date from 2019-02-01 to  2019-02-02  it display 3 item but when i used filter_by_datetime. it only filter 2019-02-02 data.


Develop APIs separately.

Fixes #704

Coverage remained the same at 100.0% when pulling 4abf522d62145dcd3f65d267faedcbbec2aa58ac on apis into de6ddaef3fe59ffe4b241c9a01ab3237a150a6c8 on master.


tf keras model.fit does not work with strings as labels

Please make sure that this is a bug. As per our [GitHub Policy]System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 18.04
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): tf 1.12, '2.0.0-dev20190503'
Python version: 3.6.7
CUDA/cuDNN version: not applicable
GPU model and memory: not applicable

Describe the current behavior
When using model.fit it throws an error:

tensorflow.python.framework.errors_impl.UnimplementedError: Cast string to float is not supported [Op:Cast] name: Cast/

Describe the expected behavior
This should work the same as when using a lower level implementation with a GradientTape (working fine, no error)
Code to reproduce the issue
    import tensorflow as tf
    import numpy as np

    images = np.random.rand(5, 108, 56, 3)
    y_pred = np.random.rand(5, 4)
    y_true = np.array(['aa', 'bb', 'cc', 'dd', 'ee'])

    dataset = tf.data.Dataset.from_tensor_slices((images, y_true))
    dataset = dataset.batch(5)
    dataset = dataset.repeat()

    model = tf.keras.Sequential([
        tf.keras.layers.Conv2D(16, [3,3], activation='relu'),
        tf.keras.layers.GlobalAveragePooling2D(),
        tf.keras.layers.Dense(4)
    ])

    def triplet_loss(y_true, y_pred):
        all_diffs = tf.expand_dims(y_pred, axis=1) - tf.expand_dims(y_pred, axis=0)
        distances = tf.sqrt(tf.reduce_sum(tf.square(all_diffs), axis=-1) + 1e-12)

        furthest_positive = tf.reduce_max(distances, axis=1)

        closest_negative = tf.map_fn(lambda x: tf.reduce_min(x),
                                     distances)

        diff = furthest_positive - closest_negative
        diff = tf.nn.softplus(diff)

        return diff

    optimizer = tf.optimizers.Adam(learning_rate=0.001)

    model.compile(loss=triplet_loss,
                  optimizer=optimizer)

    model.fit(dataset, steps_per_epoch=5, epochs=10, verbose=1)

When using a lower level implementation it works correctly:
for images, labels in dataset:
    with tf.GradientTape() as tape:
        y_pred = model(images, training=True)
        loss_value = triplet_loss(labels, y_pred)

        grads = tape.gradient(loss_value, model.trainable_variables)
        optimizer.apply_gradients(zip(grads, model.trainable_variables))
        print('iteration done')

Other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

@dmus This question is better asked on StackOverflow since it is not a bug or feature request. There is also a larger community that reads questions there.If you think we've misinterpreted a bug, please comment again with a clear explanation, as well as all of the information requested in the issue template. Thanks!

I could reproduce the issue in TF2.0.0-alpha0. Gist is here. Thanks!

This is fixed TF 2.0 nightly '2.0.0-dev20190718'
Thanks!

I am still getting this error with the same code snippet
In [6]: tf.__version__
Out[6]: '2.2.0'

Error:
UnimplementedError:  Cast string to float is not supported
	 [[node Cast (defined at <ipython-input-5-b46e9e31129b>:1) ]] [Op:__inference_train_function_978]

I think am getting the same error while using labels as strings running tf 2.2.0.  Int labels seem to work just fine:
Epoch 1/5
UnimplementedError                        Traceback (most recent call last)
 in ()
1 model.fit(train_ds,
2           epochs=5,
----> 3           steps_per_epoch = 3600//320)
8 frames
/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/engine/training.py in _method_wrapper(self, *args, **kwargs)
64   def _method_wrapper(self, *args, **kwargs):
65     if not self._in_multi_worker_mode():  # pylint: disable=protected-access
---> 66       return method(self, *args, **kwargs)
67
68     # Running inside run_distribute_coordinator already.
/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/engine/training.py in fit(self, x, y, batch_size, epochs, verbose, callbacks, validation_split, validation_data, shuffle, class_weight, sample_weight, initial_epoch, steps_per_epoch, validation_steps, validation_batch_size, validation_freq, max_queue_size, workers, use_multiprocessing)
846                 batch_size=batch_size):
847               callbacks.on_train_batch_begin(step)
--> 848               tmp_logs = train_function(iterator)
849               # Catch OutOfRangeError for Datasets of unknown size.
850               # This blocks until the batch has finished executing.
/usr/local/lib/python3.6/dist-packages/tensorflow/python/eager/def_function.py in call(self, *args, **kwds)
578         xla_context.Exit()
579     else:
--> 580       result = self._call(*args, **kwds)
581
582     if tracing_count == self._get_tracing_count():
/usr/local/lib/python3.6/dist-packages/tensorflow/python/eager/def_function.py in _call(self, *args, **kwds)
609       # In this case we have created variables on the first call, so we run the
610       # defunned version which is guaranteed to never create variables.
--> 611       return self._stateless_fn(*args, **kwds)  # pylint: disable=not-callable
612     elif self._stateful_fn is not None:
613       # Release the lock early so that multiple threads can perform the call
/usr/local/lib/python3.6/dist-packages/tensorflow/python/eager/function.py in call(self, *args, **kwargs)
2418     with self._lock:
2419       graph_function, args, kwargs = self._maybe_define_function(args, kwargs)
-> 2420     return graph_function._filtered_call(args, kwargs)  # pylint: disable=protected-access
2421
2422   @property
/usr/local/lib/python3.6/dist-packages/tensorflow/python/eager/function.py in _filtered_call(self, args, kwargs)
1663          if isinstance(t, (ops.Tensor,
1664                            resource_variable_ops.BaseResourceVariable))),
-> 1665         self.captured_inputs)
1666
1667   def _call_flat(self, args, captured_inputs, cancellation_manager=None):
/usr/local/lib/python3.6/dist-packages/tensorflow/python/eager/function.py in _call_flat(self, args, captured_inputs, cancellation_manager)
1744       # No tape is watching; skip to running the function.
1745       return self._build_call_outputs(self._inference_function.call(
-> 1746           ctx, args, cancellation_manager=cancellation_manager))
1747     forward_backward = self._select_forward_and_backward_functions(
1748         args,
/usr/local/lib/python3.6/dist-packages/tensorflow/python/eager/function.py in call(self, ctx, args, cancellation_manager)
596               inputs=args,
597               attrs=attrs,
--> 598               ctx=ctx)
599         else:
600           outputs = execute.execute_with_cancellation(
/usr/local/lib/python3.6/dist-packages/tensorflow/python/eager/execute.py in quick_execute(op_name, num_outputs, inputs, attrs, ctx, name)
58     ctx.ensure_initialized()
59     tensors = pywrap_tfe.TFE_Py_Execute(ctx._handle, device_name, op_name,
---> 60                                         inputs, attrs, num_outputs)
61   except core._NotOkStatusException as e:
62     if name is not None:
UnimplementedError:  Cast string to float is not supported
[[node Cast (defined at :3) ]] [Op:__inference_train_function_1519]
Function call stack:
train_function


unable to connect local runs to ingress api

I'm able to get the web ui working properly with these custom settings:
server:
  publicApiUrl: "https://example.com/api"
ingress:
  enabled: true
  host:
    hostname: example.com
postgresql:
  auth:
    password: "prefect"

However, when I try to connect to the API from my local laptop I keep getting failed connection errors:
RuntimeError: Cannot create flow run. Failed to reach API at https://example.com/api/.

(after running prefect config set PREFECT_API_URL=https://example.com/api/)
Is there a step that I'm missing here? There's no example yaml file in this repo's README, so it's difficult know if I'm on the right track. Seems like this user had a similar issue but no answer was ever posted.
Thanks in advance for any help 😄

So this is something I also struggled with a lot, however we did get it working.
First question is, when you say you got the web ui working, does this mean you can navigate to https://example.com/api and it shows the prefect UI? With no auth layers that need to be bypassed?
2nd: I see you haven't specified the tls flag to true? So does that mean if you go to that prefect ui url in your browser it prompts you that it's an insecure connection?

Sorry for the slow reply -- I didn't see the notification from your response!

after taking a closer look, I might not have set up the api successfully like I thought. The interface wasn't showing any errors and I could do things like adding concurrency limits etc. without any issues. So I had assumed that the full UI was working. But when I go to https://example.com/api, I get a 404 not found error. And when I go to an example endpoint like https://example.com/api/flows/count, I get a json response like this:

{"exception_message":"Invalid request received.","exception_detail":[{"loc":["path","id"],"msg":"value is not a valid uuid","type":"type_error.uuid"}],"request_body":null}

I never get any warnings in the UI, but when I inspect the console errors on the page makes it seem like redirects are getting blocked (even with tls: true and also trying with selfSigned: true):

Access to manifest at 'https://login.microsoftonline.com/...' (redirected from 'https://example.com/ico/site.webmanifest') from origin 'https://example.com' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.

Inspecting the network + ajax calls, I can see there are at least some successful calls to the api endpoints though. There's a CORS error at the end of these calls, which I'm not familiar with:

Hi @jacksund - we would definitely review a PR to the readme if you have any suggested configuration steps to help future users regarding setting up / maintaining their ingress resource!

I would gladly open a PR, but... I actually wasn't able to figure this out in the time I had 😓.
Did your team have any recommendations on how to fix the errors I'm seeing above?

@jacksund I'm not sure we can be helpful here. The configuration will be highly dependent on your infrastructure setup. We have deployed the ingress locally using a GCP load balancer & ingress resource and have it successfully working without TLS. I'm going to close this issue as it's not related to the configuration of the chart.

@jacksund For recommendations, it sounds like a browser-side blocking the requests due to cross-origin.
Does it work if you try (just temporarily) to launch your browser with these parameters ?:
chrome --disable-web-security --user-data-dir
If so, it sounds like a CORS headers config issue that you'd need to set-up. Here's two posts that may help:
https://serverfault.com/a/176729
https://medium.com/@baphemot/understanding-cors-18ad6b478e2b


fixed SKIP_RESOURCES_WAIT functionality

fixes [#295]

Thanks for contribution @serzhshakur. I will take a look in to it today.
All Best,
Jakub.

Hey @serzhshakur, it has been merged and released as<EMAIL_ADDRESS>Thanks for contribution!
All Best,
Jakub.


VonageClient.SmsClient.SendAnSmsAsync unit test: "The type initializer for 'Vonage.Configuration' threw an exception."

Trying to write an XUnit unit test to test the integration. Unfortunately VonageClient.SmsClent.SendAnSmsAsync method threw a TypeInitializationException when initialize Vonage.Configuration.

The InnerException is a MethodNotFoundException:
Method not found: 'System.Collections.Generic.Dictionary`2<System.String,System.Object> Microsoft.Extensions.Configuration.IConfigurationBuilder.get_Properties()'.

What I am missing? Is the SDK capable to run under XUnit framework?
Code to reproduce:
            VonageClient client = new VonageClient(
                Credentials.FromApiKeyAndSecret(
                    Config.ApiKey,
                    Config.ApiSecret)
            );

            // Send sms request
            SendSmsRequest smsRequest = new SendSmsRequest
            {
                From = Config.From,
                To = "your phone number",
                Text = "test message",
                Type = SmsType.Text,
                Callback = "https://<your callback url>",
                Ttl = 30000
            };

            SendSmsResponse smsResponse = await client.SmsClient.SendAnSmsAsync(smsRequest);

Hi @surexxx,
Thanks for contributing so much recently 😄
I remember you posted this one on the community Slack already, and I actually replied two months ago.
So, the SDK does run well with xUnit. It is the main testing framework used in our test project. That's definitely an odd problem you got there. I remember finding this post on SO which looks like a similar problem. You may have an older version of the DLL in the cache maybe.
Please give it a try. On my end, I'll try to reproduce the problem.

Hi @Tr00d , sorry I didn't see your comment in slack channel. I'll check it. The issue is still exists.

Hi @Tr00d ,
finally the source of the issue is discovered. There was a version mismatch between Microsoft.Extensions.Configuration (3.1.29) and Microsoft.Extensions.Configuration.Json (1.1.2). After updating the last one the same version as other (3.1.29) the issue has gone and the unit test run as expected, without any exception.


Notebook path is wrong

In the instructions, this path is not found -
https://github.com/IBM/SystemML_LinReg/blob/master/notebooks/Use-Apache-SystemML-and-Spark-for-machine-learning.ipynb

Link changed to https://github.com/IBM/SystemML/blob/master/notebooks/Machine-Learning-Using-Apache-SystemML.ipynb


Pagination limit on repeatable component items

Strapi limits the results of a repeatable component to 10 items. In the graphQL query you can define the limit, but this requires identification of those repeatable components while generating the queries, and then insert those pagination options and set it to a higher limit. I couldn't figure it out myself to add it, like this plugin already does for collectionTypes.
As a workaroud I changed the defaultLimit on the Strapi side:
#config/plugins.js
module.exports = {
  ...
  graphql: {
    config: {
      defaultLimit: 1000,
    },
  },
  ...
}

Okey, great find, I have added the pagination to the query with 1000 released on 4.3.2


Request to add Easv5 information

[Enter feedback here]
There is no Easv5 information available. Kindly add them. Thanks

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

ID: ad3e8dba-4c99-47d6-ac9f-6d110ec5a43c
Version Independent ID: 308f299a-5bb2-cf20-a97d-47dd9485d574
Content: Overview of the Azure Compute Unit - Azure Virtual Machines
Content Source: articles/virtual-machines/acu.md
Service: virtual-machines
Sub-service: sizes
GitHub Login: @mimckitt
Microsoft Alias: mimckitt

@jeethendrad
Thanks for your feedback! We will investigate and update as appropriate.

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

@jeethendrad
Thanks for your feedback! We will investigate and update as appropriate.

We are currently working on a revamp of the docs to fix this. We expect an update in the next couple weeks.
#please-close


Add ECSA report to the Publications page

https://ecsa.chokepointproject.net/

@georgiamoon Is this a report? I'm looking for more information on this site for a description.

Yes, let me find the link.

Site: https://ecsa.chokepointproject.net/
Report:
http://bookshop.europa.eu/en/european-capability-for-situational-awareness-pbKK0215595/

Thanks!

Added in #249


Add Microsoft Edge

Sorry, I will release it only when I finish PostCSS 4.2 and will start Autoprefixer 5.3 (because anyway AP has locked version for Browserslist).

Thanks

Why not publish this as 0.5.1?


Remove Series.schema.

Codecov Report

Merging #993 into master will decrease coverage by 2.45%.
The diff coverage is 44.44%.

@@            Coverage Diff             @@
##           master     #993      +/-   ##
==========================================
- Coverage   94.84%   92.38%   -2.46%
==========================================
  Files          34       34
  Lines        6519     6516       -3
==========================================
- Hits         6183     6020     -163
- Misses        336      496     +160

Impacted Files
Coverage Δ

databricks/koalas/indexing.py
94.38% <100%> (ø)
:arrow_up:

databricks/koalas/internal.py
96.38% <100%> (ø)
:arrow_up:

databricks/koalas/series.py
96.16% <16.66%> (-0.16%)
:arrow_down:

databricks/koalas/usage_logging/__init__.py
24.54% <0%> (-72.73%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
50% <0%> (-50%)
:arrow_down:

databricks/koalas/__init__.py
78.04% <0%> (-7.32%)
:arrow_down:

databricks/conftest.py
91.48% <0%> (-6.39%)
:arrow_down:

databricks/koalas/frame.py
93.84% <0%> (-2.42%)
:arrow_down:

databricks/koalas/groupby.py
89.34% <0%> (-2.05%)
:arrow_down:

databricks/koalas/namespace.py
85.71% <0%> (-1.56%)
:arrow_down:

... and 4 more

Continue to review full report at Codecov.

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


Create Introduction to Figma Part 5 - Figma Plugins.md

Description 📜
I have Created a Introduction to Figma plugins - Unsplash in Documentation
Fixes #184

Type of change 📝

[ ] Audio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Video (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev (Flutter, Java, Kotlin)
[ ] Backend Dev (Java, .NET, Python)
[ ] C/CPP
[ ] Competitive Programming
[ ] Cyber Security
[ ] DSA
[ ] Database
[ ] Datascience with Python
[ ] Datascience with R
[ ] Frontend Dev (HTML, CSS, JS)
[ ] Frontend Dev (React, Angular, Vue)
[ ] Golang
[ ] Interview Prep
[ ] Java
[ ] JavaScript
[ ] MERN
[ ] Machine Learning
[ ] Open Source
[ ] Python
[ ] Rust
[ ] Statistics
[x] UI/UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

@Debasmita-01 done with it

@Debasmita-01 review it mam

@Debasmita-01 done with it ready to merge


Suggestion: Add wall sliding

I was trying to escape dory but i kept bonking plz add a toggle or something to make wall jumps a little easier.

useless

This is a suggestion not an issue.

No, not an issue, but would be a cool feature(though I've already learned to time my a-presses like a god).

useless

not very, would be nice to add

useless

it is actually very usefull

I would hate this as a skilled player

I see.
I will most likely add a togglable option for wall sliding.


{Core} Add Context.Dafault.VS.Core.Istty and Context.Default.AzureCLI.UserAgent in telemetry

Related command

Description

We found that we may need more info logged in telemetry to help us analyze customer behavior:

Context.Dafault.VS.Core.Istty to identify whether customer is running cli interactively or automatically
Context.Default.AzureCLI.UserAgent to log the same as ARM HttpIncomingRequests table does so that we can know more about runtime env

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Core


Fix : check if is_file in getMimeType

Hello,
on my localhost i have some warning when filename are check with getMimeType function.
I added an if condition around 4 Try with mime type.

Hello,
i uploaded some SVG file from figma.
I place the module in displayFooterBefore and when i go on my frontpage, i have this error :
Warning: finfo_file(/Users/julienmiclo/WebServer/repositories/shop/modules/blockreassurance/views/img/img_perso/icon-fast-delivery.svg): Failed to open stream: No such file or directory in /Users/julienmiclo/WebServer/repositories/iello-shop/modules/blockreassurance/classes/ReassuranceActivity.php on line 187

Warning: mime_content_type(/Users/julienmiclo/WebServer/repositories/shop/modules/blockreassurance/views/img/img_perso/icon-fast-delivery.svg): Failed to open stream: No such file or directory in /Users/julienmiclo/WebServer/repositories/shop/modules/blockreassurance/classes/ReassuranceActivity.php on line 192

but all my files is in the right folder. If you check if is a file before get mime type, it's avoid the warning.
regards

Thanks @julienmiclo so if I understand

Install blockreassurance v5.1.2 on prestashop 8.1.1
Hook module to displayFooterBefore
From BO blockreassurance configuration page, upload custom SVG files to be used in blocks
Go on homepage, see error

According to AdminBlockListingController.php#L178-L201, we can use ImageManager::validateUpload instead of  ReassuranceActivity::getMimeType since PS <IP_ADDRESS>, so we can solve above issues by bumping min compat. to PS 1.7.7 and replacing getMimeType by  validateUpload completely in Release 6.0.0 .;)


Add ChannelPrimer to perform GFE refresh

Implement ChannelPrimer to establish connection to the GFE so bigtable client will reconnect to the GFE periodically before the connection gets terminated by the GFE.

As discussed offline

the implementation should be moved to EnhancedStubSettings
the ChannelProvider meddling should be deferred to the construction of EnhancedBigtableStubSettings. Otherwise the customer has to worry about the ordering of setting the channel provider and enabling channel priming


update(cmake): update cpp-httplib to 0.13.1

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind feature
/area build
What this PR does / why we need it:
Update cpp-httplib, including the latest security fixes.
Note that the only two breaking changes that are documented here https://github.com/yhirose/cpp-httplib/releases do not seem to affect the way we use the library.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: LucaGuerra
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 [LucaGuerra]

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

LGTM label has been added.  Git tree hash: 24af5aa999fd32ab81513f0d422c19701b138c3f

/milestone 0.36.0

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, LucaGuerra
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 [FedeDP,LucaGuerra]

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, jasondellaluce, LucaGuerra
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 [FedeDP,LucaGuerra,jasondellaluce]

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


added working TokenIdentifier.encode

Can be checked in the playground here https://m7sm4-2iaaa-aaaab-qabra-cai.raw.ic0.app/?tag=3426353461
Also, mintNFT should return Result with TokenIndex (as implemented by Toniq Labs)

Also, mintNFT should return Result with TokenIndex (as implemented by Toniq Labs)

This is not the case, see here. Can you provide a source?


Can't copy UserAgent request header from request to RequestTelemetry.Context.User.UserAgent

Below is a very simple implementation of an initializer for grabbing the UserAgent since it's no longer in the request telemetry object.
If we set the requestTelemetry.Context.User.UserAgent to a value, its not transferred (?) to application insights (i don't see it in log analytics), but if we set the agent to the Context.User.Id property, it does.
Is the UserAgent property no longer possible to set and see in application insights? Is there a recommended way to do this now? Or is this a bug? I know that AI no longer supports adding the UserAgent by default in the sdk, but why can't we override it?
public class UserAgentHeaderTelemetryInitializer : ITelemetryInitializer
{
    readonly IHttpContextAccessor _httpContextAccessor;

    public UserAgentHeaderTelemetryInitializer(IHttpContextAccessor httpContextAccessor)
    {
        _httpContextAccessor = httpContextAccessor;
    }

    public void Initialize(ITelemetry telemetry)
    {
        if (telemetry is RequestTelemetry requestTelemetry)
        {
            if (_httpContextAccessor.HttpContext.Request.Headers.TryGetValue(HeaderNames.UserAgent, out var userAgent))
            {
                requestTelemetry.Context.User.UserAgent = userAgent; <---- doesn't work
                requestTelemetry.Context.User.Id = userAgent;        <---- does work
            }
        }
    }
}

Environment

Netcore 6
Azure AppService
Windows
"Microsoft.ApplicationInsights.AspNetCore" Version="2.20.0"

Thanks for reviewing this.

Dupe of #558
See https://github.com/Microsoft/ApplicationInsights-Announcements/issues/3 for an explanation.
@cijothomas, I think we should add an ObsoleteAttribute on the Context.User.UserAgent property, with text that points to docs.

Its not exactly a dupe since im trying to understand why setting userid works but not useragent? I am asking for a recommendation.

I wouldn't use User.Id instead of UserAgent because the Portal has some custom UI and dashboards around "user cohorts" that expects User.Id and User.AuthenticatedId to be stable, unique user identifiers.
My recommendation is to use a custom property.

Ok, thanks. Yeah, having that the property depreciated would help future integrators. Thanks for the quick response. I guess we can close this.
For anyone who wants to do this, use this
  if (telemetry is RequestTelemetry requestTelemetry)
            {
                if (_httpContextAccessor.HttpContext.Request.Headers.TryGetValue(HeaderNames.UserAgent, out var userAgent))
                {
                    requestTelemetry.Properties.Add(HeaderNames.UserAgent, userAgent);
                }
            }

Dupe of #558 See microsoft/ApplicationInsights-Announcements#3 for an explanation. @cijothomas, I think we should add an ObsoleteAttribute on the Context.User.UserAgent property, with text that points to docs.

Thanks.
@TimothyMothra Could you add Obsolete attribute?

Dupe of #558 See microsoft/ApplicationInsights-Announcements#3 for an explanation. @cijothomas, I think we should add an ObsoleteAttribute on the Context.User.UserAgent property, with text that points to docs.

Thanks. @TimothyMothra Could you add Obsolete attribute?

I just ran into this issue as well. An Obsolete attribute with further details would be really helpful and time saving here.


fix: avoid null pointer exception

When source or target child elements not exists the $altTrans->getElementsByTagName('source')->item(0) results null and its not possible to get the property nodeValue of null.
Example alt-trans block with unexisting source:
<alt-trans origin="machine-trans">
     <target/>
</alt-trans>
<alt-trans origin="memsource-tm">
     <target/>
</alt-trans>

Hi @davidlg ,
I agree for the target, but it is supposed to raise an exception when source is not present. The source tag is mandatory.
http://docs.oasis-open.org/xliff/v1.2/os/xliff-core.html#trans-unit

Hi @Ostico,
In that case (function extractTransUnitAltTrans()), the element is not trans-unit, is alt-trans.
In alt-trans the element source is not mandatory, the only mandatory element in an alt-trans tag is target.
http://docs.oasis-open.org/xliff/v1.2/os/xliff-core.html#alt-trans
See:
Zero or one  element, followed by
Zero or one  element, followed by
One  element, followed by
Zero, one or more , ,  elements, in any order, followed by
Zero, one or more non-XLIFF elements.

Changed to:

return an exception when target is not present in an alt-trans.
not fail if source is not present in an alt-trans.

Hi @davidlg and thank you for your job.
The proposed modify makes perfectly sense, so I'll merge your PR.

Thanks @mauretto78


Funscripts don't work when using XBVR versions later than 0.4.23

When viewing videos via XBVR 0.4.24 onwards through Heresphere on a Meta Quest 3 the associated funscripts don't appear to fully load in MultiFunPlayer. The name of the scene appears without ID and the heatmap stays blank.
Using XBVR 0.4.23 everything works as it always has, the script loads with the heatmap and plays correctly.
I know there was a breaking change in XBVR that caused this and I'd generally think it would be fixed at there, however I saw on the XBVR Discord that @Yoooi0 had a PR with a potential fix and it has since been merged.
I didn't put this in Issues because it feels like maybe it's a known just undocumented issue. Just wondering if there's a chance a fix being actively worked on for a future MFP release?
Thanks!
Setup:
XBVR 0.4.23
Initface Central v2.6.0
MultiFunPlayer v1.30.2 (Patreon Build)
Heresphere 0.11.0

I don't think it is possible for HereSphere + XBVR + MFP to work at all, so dunno how you managed 0.4.23 to work.
There were two issues I fixed in XBVR 0.4.26 and 0.4.27.
And HereSphere was sending only the scene title instead of stream url which was fixed in v0.11.0.
The new HereSphere update requires MFP to update so HereSphere + XBVR + MFP should work once v1.30.3 or v1.40.0 is released.


add an importPath option for the deploy function

Is your feature request related to a problem? Please describe.
When a project depends on artifacts from external libraries and several artifacts have the same name, calling deploy with a contract name does not allow selecting which artifact to load.
Describe the solution you'd like
An optional importPath field in DeployOptionsBase which allows to retrieve the artifact by its name from a specific folder.
Describe alternatives you've considered
As a workaround, one can do:
const {getArtifactFromFolders} = require('hardhat-deploy/dist/src/utils');
// ...
const options = {
  contract: await getArtifactFromFolders(artifactName, [artifactPath]),
  // ...
};
await deploy(name, options);

did you try specifying  contract: "<path>:<contractName>" ?

It doesn't look like it will help much for my use-case, because the internal structure of the external libraries artifacts is the same:
I have these dependencies:
node_modules/projectA/artifacts/contracts/Contract.sol/
node_modules/projectB/artifacts/contracts/Contract.sol/
To use fully-qualified names, I need to have an external config similar to this:
external: {
  contracts: [
    {artifacts: 'node_modules/projectA/artifacts'},
    {artifacts: 'node_modules/projectB/artifacts'},
  ],
}

In the end, I have the same issue when deploying, still not specifying which folder to read the artifact from:
await deploy("Contract", "contracts/Contract.sol:Contract"

What I am proposing works as follows:
await deploy("Contract", {contract: "Contract", importPath: "node_modules/projectA/artifacts"});

The solution should be to use Full Qualified Name.
await deploy('src/projectA/Contract.sol:ContractName');


Add NVCC inability to compile .hip files notice and workaround

This docs change serves to clarify an error encountered when attempting to compile .hip files with nvcc, through hipcc.
e.g: As encountered by user in https://github.com/ROCm/HIP/issues/3582

This will probably need to be consolidated around a separate FAQ-rewrite PR, once that gets merged: https://github.com/ROCm/HIP/pull/3596

@MKKnorr I have revised this change per @yxsamliu's and your comments. It has also been rebased to docs/develop instead of amd-staging to account for the moved how-to/faq.md to root/faq.rst.

@MKKnorr I have revised this change per @yxsamliu's and your comments. It has also been rebased to docs/develop instead of amd-staging to account for the moved how-to/faq.md to root/faq.rst.

@MKKnorr What is the status of this PR?

Sorry, the updates of this PR slipped by me! If you rebase I'll take another look. Sorry for the delay!

Apologies, my attempt to git rebase to the head of docs/develop somehow pulled in commits from the top of docs/develop that appear "new" and prevent merging... I'll spend some time trying to un-footgun myself later this afternoon

@MKKnorr , the warning has been moved to the section entitled "hipcc detected my platform incorrectly. What should I do?".
Are any other changes needed?

@MKKnorr , the warning has been moved to the section entitled "hipcc detected my platform incorrectly. What should I do?".
Are any other changes needed?

Thanks! Nope, looks good as it is


CDTS font errors in console, rejected by FireFox

downloadable font: Failed to convert WOFF 2.0 font to SFNT (font-family: "Glyphicons Halflings" style:normal weight:400 stretch:100 src index:1) source: https://ssl-templates.services.gc.ca/app/cls/WET/gcintranet/v4_0_32/fonts/glyphicons-halflings-regular.woff2
downloadable font: rejected by sanitizer (font-family: "Glyphicons Halflings" style:normal weight:400 stretch:100 src index:1) source: https://ssl-templates.services.gc.ca/app/cls/WET/gcintranet/v4_0_32/fonts/glyphicons-halflings-regular.woff2

Glyphicons are still working in my app at least, but the console warnings are annoying...
Potentially relevant:
Proxy misconfiguration causing fonts to raise errors
https://github.com/ember-cli/ember-cli/issues/1532

@TeamDman Could you confirm this issue still exists in version 4.0.39 ? Thanks.

Still seems to be present

Font issue has been fixed in v4.0.43.
This can be confirmed via the latest sample pages.
https://ssl-templates.services.gc.ca/app/cls/WET/gcintranet/v4_0_43/cdts/samples/index.html


Use namespace imports

https://www.effect.website/docs/essentials/importing#namespace-imports
Rewrite source code manually or make a script for that.
It's for esbuild and Metro.
Next.js https://vercel.com/blog/how-we-optimized-package-imports-in-next-js

Done.


[AIRFLOW-1367][email] Pass Content-ID

Dear Airflow maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[x] My PR addresses the following Airflow JIRA

Description

[x] Here are some details about my PR, including screenshots of any UI changes:
No UI changes, just adding Content-ID in the email

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:
They already exist

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue.

@aoen

Codecov Report

Merging #2410 into master will increase coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2410      +/-   ##
==========================================
+ Coverage   69.24%   69.26%   +0.02%
==========================================
  Files         146      146
  Lines       11229    11230       +1
==========================================
+ Hits         7775     7779       +4
+ Misses       3454     3451       -3

Impacted Files
Coverage Δ

airflow/utils/email.py
97.36% <100%> (+0.03%)
:arrow_up:

airflow/jobs.py
75.35% <0%> (+0.3%)
: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 d02e8eb...57e4f8a. Read the comment docs.

LGTM


Update to allow the latest python-slugify

The hope here is that this will work perfectly and without issue.
Fixes https://github.com/microsoft/playwright-pytest/issues/159.

Thanks


Issue/#83 release annotations

Fix for DiagramGeneratorController

Hello @korzol
Could you check why we see commits that are already merged in to master?

Hello @ceckoslab
I am going to close this PR because already created new fixed PR #139


Cluster with 4 nodes but only 3 are listed under brokers section

Cluster with 4 nodes but only 3 are listed under brokers section.
In the call to /stats there is only disk usage returned for 3 brokers, the broker that is not returned in disk stats is the missing broker
There are 4 brokers returned in the json response from GET http://localhost:8080/api/clusters/sensor/brokers

Set up
App version: UI for Apache Kafka
vv0.4.0(
521ba0c
)
Kafka version: 3.2.0 in kraft mode
Screenshots
https://cdn.discordapp.com/attachments/897805035122077719/1010777595882782750/unknown.png

relates to https://github.com/provectus/kafka-ui/issues/2738, should be fixed when https://github.com/provectus/kafka-ui/pull/2903 merged


DLL function does not see INI file path

Hello,
Thank you for this amazing library.
I am using a DLL, that reads a .INI file from the same folder as that of DLL's. I am able to call the function. However the function should read the INI file and return first value of first key. But it keeps saying that it was not able to read the INI file.
If I call the same function in a C++ code, it works okay.
Things I have tried:
1 - I tried adding the path to the DLL in system environment variable as 'process.env['PATH'].
2 - I tried changing the path of the dll at different locations within a project directory.

I was able to solve the issue.
The vendor's dll was reading INI files relative to its  path. Now when the DLL was compiled, it was neccssary that any process that uses it must be in same directory. However when I was using it with Electron, the main process path was different (as I put dll folder in extraFiles config). Hence it was unable to read the INI file.
I moved the dll and INI file under src folder and it works perfectly.


Outsized toolbar when running Python scripts in a smaller window

When a VSCode window is resized to be smaller, all of the toolbar items are retained at the expense of crowding out actual Python output, which seems like a mistake.  In particular, the screen real estate allocated to relatively unimportant information like the current Python kernel label seems far too large.

Is it possible to hide these altogether, or at least make them smaller?

Sorry about the toolbar being too big. There's no way to change it at the moment. We'll consider it in a future revision though.

Yes, we will consider in a future version as there are plans to somewhat redesign part of the interactive window.

Thanks, that would be great if there was a rethink of this. As you can see, its not difficult to cause the toolbar to take up most of the available space:

Hey there, this constitutes a serious accessibility problem for low vision users such as myself. I generally use the extension full screened to better read the text, but I am limited by what I can have on my screen besides the toolbar.
Some potential suggestions that would improve my experience, in order of significance

Allow the user to fully hide the toolbar from the settings, similar to "jupyter.mainToolbar.visible": false
Stop the little icons on the left side of the toolbar from  line wrapping
Allow the toolbar to be resized with the mouse

Thank you for your time

@relishyeah The notebook UI is actually being redesigned. You might want to try the insiders  version of VS Code. It does not have this same problem.
The interactive window does still have this problem though.


Add dbmaintenance command to update adviser contact email addresses

Issue number: N/A
Description of change
Adds a command to update the contact_email field for advisers.
Checklist

[x] Have any relevant search models been updated?
[x] Have any relevant fixtures (fixtures/test_data.yaml) been updated?
[x] Have any relevant select-/prefetch-related field lists in the views and search apps been updated?
[x] Has the admin site been updated (for new models, fields etc.)?
[x] Has the README been updated (if needed)?

Codecov Report

Merging #826 into develop will increase coverage by 0.01%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop     #826      +/-   ##
===========================================
+ Coverage    96.54%   96.56%   +0.01%
===========================================
  Files          154      155       +1
  Lines         4518     4541      +23
  Branches       441      443       +2
===========================================
+ Hits          4362     4385      +23
  Misses          96       96
  Partials        60       60

Impacted Files
Coverage Δ

...anagement/commands/update_adviser_contact_email.py
100% <100%> (ø)

datahub/dbmaintenance/utils.py
100% <100%> (ø)
: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 f5bda04...02d64e1. Read the comment docs.


UserSolutions: Some updates regarding solutions upload in UserSolutionSubmissionView

It is not allowed to submit solution with empty (school, school_class, class_level) fields
It is ok to correct or do any admin work with uploaded solutions (even if above profile fields are empty)
User gets warning to check the result of conversion, when uploading doc and docx files.

Thanks!


Multiple simulators on Jenkins Instance when running two simultaneous jobs

I am using Fastlane on a Jenkins set up with one OSX slave.
I have various jobs that I want to run simultaneously. However when two jobs are running that require use of the simulator to run UnitTests (no UI tests), one of the jobs fails. I believe this is because both jobs can't use the simulator at the same time.
Has anyone managed to overcome this limitation (using Fastlane)?
I heard of FBSimulatorControl, but I don't think that tool is integrated in Fastlane, and I am not sure it should be. But what are your thoughts on this?
(I am sorry if fastlane's repo is not the best place for this question)

Hi @beloso
fastlane doesn't have a feature like this one.
Running multiple simulators in parallel is not something that's possible out-of-the-box.
Apple has done a lot of work with regards to managing simulators, but no progress in regards to having multiple instances running in parallel.
I heard about FBSimulatorControl just few months ago, but haven't had a chance to try it out.
They don't even make it easy to install (no Homebrew formula or Ruby gem).
I guess a Fastlane plugin is the way to go.
But someone would have to create a plugin for that.

There is a brew formula for fbsimctl
https://github.com/facebook/FBSimulatorControl/blob/master/fbsimctl/README.md
I am wondering this because right now we have a mac pro that can only process one iOS Job at a time due to this limitation, which is kind of a waste :/
When I have some time on my hands I'll take a deeper look at fbsimctl. I just had to ask if anyone was going though this, and if there are known workarounds.


Neural networks for high quality textures generation

Is your feature request related to a problem? Please describe.
Some textures look bad in the game, like ground, footpaths, water, etc.
Describe the solution you'd like
Use NN for upsampling and maybe for generation of the realistic look. Like it was for Duke Nukem face.

Additional context
You made a great work! I'm very impressed! Thank you, guys!

Hi @makseq, thanks for the praises.
I actually had that idea in mind at the very beginning of the project, but never got around to try it out.
I can see two potential issues that would need to be addressed for implementing this:

The original textured are stored in a specific format, in a texture atlas with a color palette, we're currently reading that format directly in our game engine and making use of the palette. The loading path for various type of textures in the game (static objects, characters, ground textures) is different, and so is the way the atlas work. We need to unpack all that to known formats and implement different loading code paths to load high-res versions. It's a bit more involved than it looks like.
I don't think we want to go for a realistic style here, one specificity of this project is that we're trying to preserve the art style of the original as much as possible. To do that with upscaling, I think we'd need to train a custom model with a carefully curated dataset coherent with the sort of style we want to achieve (roughly: colorful/candy/cartoonish/comic-like style).


adapt CSV parsing to new data model

For each work the reports in CSV format collect total counts for every source. For three sources (mendeley, counter, pmc) we need additional information (readers, html, and pdf counts, respectively). Before Lagotto 4.0 this was done using a to_csv method in the respective model. With Lagotto 4.0, and the switch to store all sources data in MySQL instead of CouchDB, this functionality is broken.

Relevant code:

Reportmodel: https://github.com/articlemetrics/lagotto/blob/master/app/models/report.rb
Custom to_csv methods in [mendeley[(https://github.com/articlemetrics/lagotto/blob/master/app/models/sources/mendeley.rb), counter, and pmc. The to_csv code for counter and pmc is no longer in the master branch.
rake tasks to generate reports: https://github.com/articlemetrics/lagotto/blob/master/lib/tasks/report.rake
report generation is triggered via cron: https://github.com/articlemetrics/lagotto/blob/master/config/schedule.rb and https://github.com/articlemetrics/lagotto/blob/master/lib/tasks/cron.rake

@mfenner, here are some observations / assumptions I made today exploring the code...
1: All source models (except Mendeley) inherit their to_csv functionality from the base Source model. Given that all source data moved from CouchDB to MySQL in the 4.0 release I believe this is why all source CSV reports are broken and that fixing Source#to_csv will resolve that particular issue (except for Mendeley).
2: Three individual source CSV reports – Mendeley, Counter, Pmc – need additional information that the other source CSV reports do not require. My assumption here is that once generic source CSV reports are working again (e.g. Source#to_csv) then these three reports can be updated to include the new fields: readers, html, counts.
3: Do you want the new columns (readers,html,counts) to be pushed up and included in the aggregate ALM report? If so, I'll plan on updating that the code that generates the ALM report to handle that.
Let me know if I'm heading down the right track for the intended goals of this issue or if I'm missing anything. Thanks!

1: Yes, fixing the CSV generation should be straightforward. Two other sources (pmc, counter) also have a to_csv, but I have removed it. I linked the revisions above that still contain to_csv.
2: Yes.
3: Yes, the aggregate report used to have separate columns for html, pdf, etc. where it mattered. Forgot to include a link to a sample report: http://figshare.com/articles/Cumulative_PLOS_ALM_Report_February_2014/1189396
The only other comment I would make is that I found the code used for report generation always a bit obscure, e.g. the giant SQL call in Report.from_sql (https://github.com/articlemetrics/lagotto/blob/master/app/models/report.rb#L33-L41). I think covering this in tests works fine, but some of the database calls take rather long, and there was for example an issue with timeouts of CouchDB calls (no longer relevant). In the calls to external APIs I was able to break large queries into chunks of 200 or 1000 that are then handled by Sidekiq in parallel (the latter for example in https://github.com/articlemetrics/lagotto/blob/master/app/models/imports/crossref_import.rb. Not sure how much refactoring would be required to do something similar with the CSV report.

The only other comment I would make is that I found the code used for report generation always a bit obscure, e.g. the giant SQL call in Report.from_sql (https://github.com/articlemetrics/lagotto/blob/master/app/models/report.rb#L33-L41)

Yeah, I can see that. I think there's an intermingling of report-related concepts in the Report model. It took me a little bit to mentally separate out that the class-level reports generated for the CSV aren't related to the kinds of reports that the Report model represents and that the Report class has other unrelated service-like methods (all of the send_xxx methods for kicking off mailers).
As I work on getting the CSV reports back in place I'm going to put a little thought on how to refactor and further the separation of those concepts.

Great.

@mfenner, I was looking at the reports.json from 691d4a645d20f3625a94868e633120f64bd1f716 to make sure I had an accurate understanding of what those views were doing.  When generating a format specific CSV source report does the event's date and count sums correspond with the year/date and sums in Month model?

Yes, the Month model replaced this information from CouchDB.
We are actually several different reports (see https://github.com/articlemetrics/lagotto/blob/master/lib/tasks/report.rake). The only one that we ever made public, and is the most important one is the summary report, generated by the combined_stats rake task.
Part of this is historic. Before there was Ruby code these reports were generated by a combination of Perl, R and some manual merging. The intermediate reports are useful, but strictly speaking no longer needed for the combined_stats report, as we can get all the required info from the retrieval_statuses table now (after adding columns for readers, html and pdf).

The core work on this is almost completed (see PR #351). I'm nearing completion on updating report:combined_stats and report:combined_private_stats rake tasks.  That will conclude all of the reports being generated using data from MySQL.
I haven't looked at the scheduling code yet, but I'm anticipating that will either not need to change at all or it will need to change very minimally.


Expose PhysicalBone::get_bone_id() to c#/gdscript

Making it possible to get the bone index of a PhysicalBone node.

can be merged

Great! Thanks.


[Question]

[READ] For Firebase Unity SDK question, please report to Firebase Unity Sample
Once you've read this section and determined that your issue is appropriate for this repository, please delete this section.
[REQUIRED] Please fill in the following fields:

Pre-built SDK from the website or open-source from this repo: _____
Firebase C++ SDK version: main
Main Firebase Components in concern: Database (Auth, Database, etc.)
Other Firebase Components in use: _____ (Auth, Database, etc.)
Platform you are using the C++ SDK on: Arm Linux (raspberry Pi) (Mac, Windows, or Linux)
Platform you are targeting: raspberry Pi (iOS, Android, and/or desktop)

[REQUIRED] Please describe the question here:
Hi, I am trying to access database from raspberry pi running Linux (Armv7l). Can some one provide guide what changes need to be made for the c++ api to work on that?

And to further describe my encounter question: when running the test_linux.sh on the raspberry pi, I encounter the following error:
/usr/bin/ld: CMakeFiles/firebase_app_scheduler_test.dir/scheduler_test.cc.o: in function `firebase::compat::Atomic<unsigned long long>::fetch_sub(unsigned long long)':
scheduler_test.cc:(.text._ZN8firebase6compat6AtomicIyE9fetch_subEy[_ZN8firebase6compat6AtomicIyE9fetch_subEy]+0x40): undefined reference to `__atomic_fetch_sub_8'
/usr/bin/ld: ../libfirebase_app.a(scheduler.cc.o): in function `firebase::compat::Atomic<unsigned long long>::fetch_add(unsigned long long)':
scheduler.cc:(.text._ZN8firebase6compat6AtomicIyE9fetch_addEy[_ZN8firebase6compat6AtomicIyE9fetch_addEy]+0x40): undefined reference to `__atomic_fetch_add_8'
/usr/bin/ld: ../libfirebase_app.a(scheduler.cc.o): in function `firebase::compat::Atomic<unsigned long long>::store(unsigned long long)':
scheduler.cc:(.text._ZN8firebase6compat6AtomicIyE5storeEy[_ZN8firebase6compat6AtomicIyE5storeEy]+0x50): undefined reference to `__atomic_store_8'
collect2: error: ld returned 1 exit status
gmake[2]: *** [app/tests/CMakeFiles/firebase_app_scheduler_test.dir/build.make:109: app/tests/firebase_app_scheduler_test] Error 1
gmake[1]: *** [CMakeFiles/Makefile2:8684: app/tests/CMakeFiles/firebase_app_scheduler_test.dir/all] Error 2
gmake[1]: *** Waiting for unfinished jobs....

Seems to be an issue with lib atomic. Any suggestions on how to fix this?

I think this may be an issue that current build configurations for linux only support x86 and x64 architectures.
It is theoretically possible, but it is not clear what would be needed to support linux arm architectures.


FileNotFoundError: [Errno 2] No such file or directory: '/Users/bingqi/PycharmProjects/tf-faster-rcnn-rddc/data/VOCdevkit2007/VOC2007/Annotations/test_Muroran_00398.xml'

The website of original dataset cannot be open. There's no xml file for test dataset in the converted dataset.

The challenge's homepage has been closed, so you can‘t get the official score now.
The original Road Damage Dataset can be downloaded here.

The website of original dataset cannot be open. There's no xml file for test dataset in the converted dataset.

The challenge's homepage has been closed, so you can‘t get the official score now.
The original Road Damage Dataset can be downloaded here.

here

Thank you very much! Do you use the same pictures for both training and testing? I've noticed that there are some names in "test.txt" are not included in your "JPEGImage" file as well as annotations, such as "test_aug_Sumida_00143".

here

Thank you very much! Do you use the same pictures for both training and testing? I've noticed that there are some names in "test.txt" are not included in your "JPEGImage" file as well as annotations, such as "test_aug_Sumida_00143".

Data augmentation techniques are used during training, and original test images are used to get the submission.

here

Thank you very much! Do you use the same pictures for both training and testing? I've noticed that there are some names in "test.txt" are not included in your "JPEGImage" file as well as annotations, such as "test_aug_Sumida_00143".

Data augmentation techniques are used before training, and original test images are used to get the submission.

here

Thank you very much! Do you use the same pictures for both training and testing? I've noticed that there are some names in "test.txt" are not included in your "JPEGImage" file as well as annotations, such as "test_aug_Sumida_00143".

Data augmentation techniques are used before training, and original test images are used to get the submission.

Thank you very much for your reply!

I found the training data augmentation cannot work due to some issues. I have changed the paths but the error is like this:
KeyError                                  Traceback (most recent call last)
 in
27 count = 0
28 for im_name in images:
---> 29     for bbox in anno_dict[im_name]:
30         if int(bbox[0]) in [3,4,7]:
31             do_aug = 1 if do_aug < 1 else do_aug # 2 times
KeyError: '/Users/bingqi/PycharmProjects/tf-faster-rcnn-rddc/data/VOCdevkit2007/VOC2007/JPEGImages/train_Adachi_00001.jpg'


No or() in yml conditions

Is there no or(arg1, arg2) for conditions in yml?
I did this:
l195   condition: and(succeeded(), or(eq(variables['Queue.Ssp'], 'ON'), ne(variables['Build.Reason'], 'PullRequest'))
and it gives me:
/azure-pipelines.yml: (Line: 195, Col: 3, Idx: 7008) - (Line: 195, Col: 3, Idx: 7008): While scanning a simple key, could not find expected ':'.
Queue.Ssp is currently undefined.

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

ID: 3f151218-9a11-0078-e038-f96198a76143
Version Independent ID: 09c4d032-62f3-d97c-79d7-6fbfd89910e9
Content: Conditions - Azure Pipelines
Content Source: docs/pipelines/process/conditions.md
Product: devops
Technology: devops-cicd-process
GitHub Login: @juliakm
Microsoft Alias: jukullam

Syntax error. Missing ). Sorry.


[Fix] Allow merging of resolved incidents into open incidents

Summary
This PR closes #140 where users were not able to merge resolved incidents into open incidents.
We have also included the following changes:

Updated logic to activate merge button for at least one selected incident; this is now consistent with main PD UI
Updated logic to remove merged incident from incident table using unlink_log_entry
Added component test coverage for parent component IncidentActionsComponent

Before Fix

After Fix

Only one test failed due to container performance - everything else passed - merging.


Update eslint-config-recommended

This update fixes building errors with the latest Phenomic installation.
Fixes #838

I would like to get a failure on CI before "fixing" this. Thanks for this anyway as I was missing it.

Cool!

First things first https://github.com/yannickcr/eslint-plugin-react/pull/925 :)

Closed by #841


[bug] incorrect driving instructions

driving directions are wrong for this location.
https://main--hubblehomes-com--aemsites.hlx.page/new-homes/idaho/boise-metro/caldwell/mason-creek

fixed


Adjustment for rails 5.2.0

Hi, Austin. Thanks for the great tutorial.
I found one adjustment for your tutorial. Since rails 5.2.0 doesn't generate secret.yml but credentials.yml.enc, the HMAC_SECRET in app/lib/json_web_token.rb should be
HMAC_SECRET = Rails.application.credentials.read
If the HMAC_SECRET still using secret_key_base, it will throw status 500 error with exception:
"#<TypeError: no implicit conversion of nil into String>
This adjustment for rails 5.2.0 only for this time. Again, I thank you for your great tutorials.

Heya @thekingisme Cheers for reaching out! Brilliant shout - would you be Ok to raise A PR for this? I'm looking to schedule some time to update the tutorial to Rails 5.2.0 - it can be merged then. 🙂

https://github.com/akabiru/todos-api/pull/28

https://github.com/akabiru/todos-api#please-note


KeyError when querying APASS catalogue

I have been running this package for a little while now and never had any issue with querying the catalogues. However, now when I run the code it goes through the TYCHO2 function alright, but then I get 'KeyError: 'recno'' when it tries to run through the APASS catalogue. I have pasted the full error below:
KeyError                                  Traceback (most recent call last)
Cell In[2], line 6
3 dec = -4.3371428022
4 gaia_id =<PHONE_NUMBER>984210304
----> 6 s = Star(starname, ra, dec, g_id=gaia_id)
File ~/anaconda3/lib/python3.11/site-packages/astroARIADNE-1.1.1-py3.11.egg/astroARIADNE/star.py:182, in Star.init(self, starname, ra, dec, g_id, plx, plx_e, rad, rad_e, temp, temp_e, lum, lum_e, dist, dist_e, Av, Av_e, offline, mag_dict, verbose, ignore, dustmap)
177 if verbose:
178     print(
179         colored('\t\t*** LOOKING UP ARCHIVAL INFORMATION ***',
180                 c)
181     )
--> 182 lib = Librarian(starname, self.ra, self.dec, g_id=self.g_id,
183                 mags=self.get_mags, ignore=ignore)
184 self.g_id = lib.g_id
185 self.tic = lib.tic
File ~/anaconda3/lib/python3.11/site-packages/astroARIADNE-1.1.1-py3.11.egg/astroARIADNE/librarian.py:168, in Librarian.init(self, starname, ra, dec, radius, g_id, mags, ignore)
166 if mags:
167     self.gaia_query()
--> 168     self.get_magnitudes()
169     idx = self.used_filters >= 1
170     self.used_filters[idx] = 1
File ~/anaconda3/lib/python3.11/site-packages/astroARIADNE-1.1.1-py3.11.egg/astroARIADNE/librarian.py:328, in Librarian.get_magnitudes(self)
326     continue
327 if c == 'APASS':
--> 328     self._get_apass(current_cat)
329     continue
330 elif c == 'Wise':
File ~/anaconda3/lib/python3.11/site-packages/astroARIADNE-1.1.1-py3.11.egg/astroARIADNE/librarian.py:613, in Librarian._get_apass(self, cat)
611 def _get_apass(self, cat):
612     print('Checking catalog APASS')
--> 613     mask = cat['recno'] == int(self.ids['APASS'])
614     self._retrieve_from_cat(cat[mask], 'APASS')
File ~/anaconda3/lib/python3.11/site-packages/astropy/table/table.py:2093, in Table.getitem(self, item)
2091 def getitem(self, item):
2092     if isinstance(item, str):
-> 2093         return self.columns[item]
2094     elif isinstance(item, (int, np.integer)):
2095         return self.Row(self, item)
File ~/anaconda3/lib/python3.11/site-packages/astropy/table/table.py:267, in TableColumns.getitem(self, item)
256 """Get items from a TableColumns object.
257
258 ::
(...)
264   tc[1:3] # <TableColumns names=('b', 'c')>
265 """
266 if isinstance(item, str):
--> 267     return OrderedDict.getitem(self, item)
268 elif isinstance(item, (int, np.integer)):
269     return list(self.values())[item]
KeyError: 'recno'
Any help on why this might be occuring now after I've already run it fine a few times would be greatly appreciated.

I noticed that this happens with only some of the stars I query result in this issue. It looks like the APASS9 catalog on Vizier doesn't contain recno (perhaps it did in the past, as I did come across in on a different mirror but no longer see it there).  My fast workaround was to implement a try except, so that the code can continue to run if there's an issue querying APASS9:
def _get_apass(self, cat):
print('Checking catalog APASS')
try:
# Attempt to access 'recno'
mask = cat['recno'] == int(self.ids['APASS9'])
print("Using 'recno' for APASS9 matching.")
except KeyError:
# 'recno' column is missing; skip APASS9 processing for this star
warning_msg = f"Warning: 'recno' column not found in APASS9 for star {self.starname}. Skipping APASS9 magnitudes."
print(warning_msg)
return
self._retrieve_from_cat(cat[mask], 'APASS')
This seems to work for me. I haven't made a request to merge into the main with this because it's not really a resolution to the issue and perhaps the developers will get a chance to update the code in a more suitable way. If I do have some time to get to a better solution, I'll update this.

I also noticed this behavior on targets that did not show this error before, even though the query did not find APASS data. I installed ARIADNE on a different system and get the same error.
It seems  recno is not an APASS id but a Vizer one, and that unfortunately did not lead to consistent results. As @norrisryan noted, it seems to have been removed.
https://stellarphot.readthedocs.io/latest/api/stellarphot.apass_dr9.html
https://www.aavso.org/apass-9-10-record-identifier

Indeed, Vizier took out the recno column. I pushed an update that ignores APASS until I have time to properly resolve this.


qwen 预训练模型支持预处理

PR types

Function optimization
PR changes

APIs
Description

千问模型预训练脚步缺少支持修复

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: wangyongqin01:x: ruisunycwangyongqin01 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

重复提交,重新申请