gfile.Glob - Recursive in Windows, not in Unix

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes.
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10 and macOS High Sierra 10.13.4
TensorFlow installed from (source or binary): pip3 install --upgrade tensorflow
TensorFlow version (use command below): 1.8.0
Python version: 3.6.5
Bazel version: N/A
CUDA/cuDNN version: N/A
GPU model and memory: N/A
Exact command to reproduce: In source code

Describe the problem
gfile.Glob() works recursively in Windows, but not in Unix-based systems.
Ideally, they should work the same way.
Source code
Stand in a root folder, create subfolder and add a image.jpg in the subfolder.
Then start python and run:
from tensorflow.python.platform import gfile
print(gfile.Glob("./*.jpg"))

In Windows, you will get [".\\subfolder\\image.jpg"].
In Unix, you will get [].

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce

@tatatodd , I think this is a duplicate of #19378 (although I think this is a better name for the issue, because I couldn't find the other one, @karmel pointed this issue out in another issue - #20559 ).

Nagging Assignee @tatatodd: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @tatatodd: It has been 29 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @tatatodd: It has been 50 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @tatatodd: It has been 65 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @tatatodd: It has been 80 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @tatatodd: It has been 95 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @tatatodd: It has been 110 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @tatatodd: It has been 125 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @tatatodd: It has been 140 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

I can now see the output, and gfile.Glob has been moved under tf.io.gfile.glob, check the working gist here for reference.


[DESIGN ONLY] Accessibility issue of the progress in How it works

Issue: Gray not strong enough.

Increased the size of the active dot and made the inactive dots darker to be WCAG 2 AA compliant.

I'm moving forward with this. I created a dedicated ticket for dev: https://github.com/project-vagabond/covid-green-app/issues/351


Return content_id in /api/organisations response

This endpoint is called by panopticon to update it's database for the organisation tags. It currently doesn't contain content_ids. We need those to migrate to the new tagging architecture.
https://trello.com/c/1JkpLaTD/421-add-content-id-to-organisation-tags-in-panopticon

The details hash contains content_ids already and they're used by Signon and Transition to identify organisations (and maybe in other apps too). The commit message isn't quite accurate because of this.
Do you need the content ids at the top level rather than in the details hash for tagging?

@jennyd thanks! I missed that one.


[Type checker] Eliminate the unused "resolve type repr" typecheck request kind.

... and add a still-crashing test case so it's easy to track.

@swift-ci please smoke test and merge


Performance of modify large files

When I create a large file (about 500kB) in LFS and then modify some bytes in this file, the LFS takes too long time (about 4 seconds) to complete it, it's really a long time for most of the bare-metal systems I think, is there any way to improve it?
Here is my config:
static const struct lfs_config lfs_cfg =
{
.read				= lfs_low_read,
.prog				= lfs_low_prog,
.erase				= lfs_low_erase,
.sync				= lfs_low_sync,
.block_size			= 512,
.block_count		= 8192,
.read_size			= 128,
.prog_size			= 256,
.cache_size			= 512,
.lookahead_size		= 16,
.block_cycles		= 300,
};

Hi @AzenkChina, thanks for raising an issue.
Right now LittleFS behaves very poorly with random-writes. Random writes effectively end up rewriting the rest of the file after the modified data.
More info here:
https://github.com/littlefs-project/littlefs/issues/27
Unfortunately there's not really a workaround other than restructuring your data to avoid random writes, either by appended data like a log, or by splitting the data up into multiple files.

Hi @AzenkChina, thanks for raising an issue.
Right now LittleFS behaves very poorly with random-writes. Random writes effectively end up rewriting the rest of the file after the modified data.
More info here:
https://github.com/littlefs-project/littlefs/issues/27
Unfortunately there's not really a workaround other than restructuring your data to avoid random writes, either by appended data like a log, or by splitting the data up into multiple files.


Add support for Windows

Solves #5
Screenshot:

Prerequisites:

BurntToast notification plugin for Powershell
fish running under WSL (Windows Subsystem for Linux)
Windows 10 (since only 10 has WSL)

@franciscolourenco Should be good to go now.

@ammgws thanks! Didn't test, don't use windows, do you want to own the windows platform?

Yeah I can keep tabs on it if stops working.


java.lang.UnsatisfiedLinkError crash

Describe the bug
This might be a duplicate for https://github.com/talsec/Free-RASP-Flutter/issues/59. The crash happened in freeRASP 4.0.0, but I haven't found a mention in the changelog for 5.0.x that it is fixed, so I'm reporting the issue.
For some users, the app crashed with the following stacktrace:
java.lang.UnsatisfiedLinkError: No implementation found for java.lang.String[] com.aheaditec.talsec_security.security.Natives.b() (tried Java_com_aheaditec_talsec_1security_security_Natives_b and Java_com_aheaditec_talsec_1security_security_Natives_b__)
	at com.aheaditec.talsec_security.security.Natives.b(Native Method)
	at com.aheaditec.talsec_security.security.Natives.i(SourceFile:1)
	at com.aheaditec.talsec.security.e1.b(SourceFile:3)
	at com.aheaditec.talsec.security.b1.t(SourceFile:1)
	at com.aheaditec.talsec.security.b1.H(SourceFile:5)
	at com.aheaditec.talsec.security.b1.p(SourceFile:1)
	at com.aheaditec.talsec.security.a1.run(R8$$SyntheticClass:5)
	at com.aheaditec.talsec.security.e.b(SourceFile:9)
	at com.aheaditec.talsec.security.b1.y(SourceFile:6)
	at com.aheaditec.talsec.security.b1.a(SourceFile:2)
	at com.aheaditec.talsec.security.l2.e(SourceFile:31)
	at com.aheaditec.talsec.security.l2.a(SourceFile:1)
	at com.aheaditec.talsec.security.k2.run(R8$$SyntheticClass:5)
	at java.lang.Thread.run(Thread.java:919)

To Reproduce
Unfortunately, I can not reproduce the crash locally.
Expected behavior
The app not crashes.
Please complete the following information:

Device: Xiaomi Nexus 6P
OS version: Android 10
Version of freeRASP: 4.0.0

Hi!
We are aware of this issue and we working on it. Fix should be available on next release.

Hello @olexale ,
we have inspected a similar issue and provided a partial solution and suggested a few more solutions, which needs to be tested. You can check it out in this issue: https://github.com/talsec/Free-RASP-Android/issues/18
Kind regards,
Talsec team

Closed as resolved.
Issue was resolved since freeRASP 6.0.0
If issue persists or reoccurs, please reopen this issue.


xboxdrv: update url

Motivation for this change
url changed
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

Wrong branch. Please open against master. We can backport from there.

I have no idea why github created the PR against that brach. That's not intendet. What a mess.


Active State

Active State added

I think this is getting too confusing: I suggest that you use the existing

get_connection_state () or is_connected()
get_audio_state ()

depending on what you want to achieve...

is need to be sure this is done

  a2dp_sink.stop();
  a2dp_sink.end();

get_connection_state () and is_connected() is the same I think.
bool IsA2DPActive() {
	if (a2dp_sink.isActive() == true) {
		return true;
	}
	else
	{
		return false;
	}
}
bool IsA2DPConnected() {
	if (a2dp_sink.is_connected() != IntConnected) {
		IntConnected = a2dp_sink.is_connected();
		if (IntConnected == 0) {
			//Serial.println("Not Connected");
		}
		if (IntConnected == 1) {
			Serial.println("Connected");
		}
	}
	return IntConnected;
}
bool IsA2DPPlaying() {
	bool BlnPlaying = false;
	if (a2dp_sink.get_audio_state() != IntStat) {
		IntStat = a2dp_sink.get_audio_state();
		if (IntStat == 0) {
			Serial.println("Stop");
			//Serial.println(IntBufferCounter);
			//for (int i = 0; i < IntBufferCounter;i++) {
			//	if (IntValues[i] != 0) {
			//		Serial.println(IntValues[i]);
			//	}
			//}
			//memset(IntValues, 0, sizeof(IntValues));
			//IntBufferCounter = 0;
		}
		if (IntStat == 2) {
			Serial.println("Play");
			BlnPlaying = true;
		}
	}
	return BlnPlaying;
}

I want to close down the esp-a2dp library to start the player from audiotools

you mean folowing this example...
if (a2dp_sink.get_audio_state()==ESP_A2D_AUDIO_STATE_STARTED){
I will try it thursday. Please do not close this yet.

I am still not sure if I understand the problem correctly: If you want to shut down A2DP, I suggest to just call end() w/o calling stop() first...

Maybe I have to add more resistors like I did with an other project with displays before. I nothest that the sd drive doesn't work anymore after reboot with multiple devices on the same spi.

Can't resolve the confilicts with the changes on my code lines. I am forced to undo all my changes I have made.


Test case for Create a product

As a Developer.
I need   a test case for the function, creating a product
So that  I can measure if this function works correctly
Details & Assumptions:

create a product(name="iPhone", category="phone", amount=1, description="the latest iPhone 13", status="Status.Good")
assert product.name is iPhone
assert product.category is phone
assert product.amount is 1
assert product.description is the latest iPhone 13
assert product.status = Status.Good

Acceptance Criteria:
All the assertions pass
Given [some changes to the source code]
When [use this test case to validate]
Then [I can measure if the changes can pass the test or not]

Close reason: Create products issue already have related test case issue


Read input from stdin

Probably needs documenting but I leave it up to you to tell me what to add :).
Fixes #3.

Thanks :)


Model urls wrong in https://github.com/zhanghang1989/detectron2-ResNeSt

The cascade_mask_rcnn_resnest50 model is :https://hangzh.s3.amazonaws.com/detectron/mask_rcnn_ResNeSt_50_FPN_syncBN_1x-f442d863.pth

Here are some models of link errors：

Cascade_RCNN_ResNeSt_200
mask_cascade_rcnn_ResNeSt_50_FPN
3.mask_cascade_rcnn_ResNeSt_101

Thanks for pointing typos out. We have updated the urls to models of mask_cascade_rcnn_ResNest_50(101)
As for (1), We will update the model later.

Thanks for your update!


[Snyk] Security upgrade ruby from 3.2.2-slim-bullseye to 3.3.2-slim-bullseye

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to fix 4 vulnerabilities in the dockerfile dependencies of this project.
Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Snyk changed the following file(s):

Dockerfile

We recommend upgrading to ruby:3.3.2-slim-bullseye, as this image has only 70 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Out-of-bounds Write SNYK-DEBIAN11-PERL-6085272
  614

Information Exposure SNYK-DEBIAN11-GNUTLS28-6062102
  514

Out-of-bounds Read SNYK-DEBIAN11-TAR-3253527
  514

Integer Overflow or Wraparound SNYK-DEBIAN11-ZLIB-6008961
  500

Integer Overflow or Wraparound SNYK-DEBIAN11-ZLIB-6008961
  500

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

Do not expect to be able to get this to build due to SQLite issue. So for now we stay below 3.3 for Ruby


Routes broken after updating to 0.4.0

After updating Lotus to 0.3.1 (lotus-router to 0.4.0) I am getting Lotus::Routing::InvalidRouteException: HttpRouter::TooManyParametersException in all my routes
This is the configuration of my current routes:
https://gist.github.com/darwinrc/c7a4457ffe27de573d25

@darwinrc can you post a backtrace?

Here's the trace: https://gist.github.com/darwinrc/c1ee0a291128784a56c5

And here's my lotus routes: https://gist.github.com/darwinrc/1db5c0ff3c7aca3ec235

@darwinrc can you paste this line? /home/darwin/dev/kommit/mobijob/mobijob-backend/apps/web/views/application_layout.rb:21:in new_job_path'`

Web::Routes.path(:jobs, :new)

@darwinrc try Web::Routes.path(:new_job), and if you updated lotus gem, you can use routes helpers in view. In your application.rb there is a view.prepare block:
view.prepare do
  include Lotus::Helpers
end

then this module is injected https://github.com/lotus/helpers/blob/master/lib/lotus/helpers/routing_helper.rb
and you can replace Web::Routes.path(:new_job) with routes.new_job_path
The same for controllers, but this behaviour is added by default https://github.com/lotus/lotus/commit/1040a996bdab8762aebcbd293b9cb1d2bcdaac65
PD: please update your code with this new syntax

I made the changes and it worked! I called it with routes.new_job_path. I'm proceeding with the update then.
Thanks a lot, @AlfonsoUceda

@darwinrc I'm closing this. @AlfonsoUceda Thank you :+1:


Complete aliases on tab

I'd be great if tab would autocomplete aliases.

This can be resolved by setting clink's exec_match_style option default value to 1 or 2.
This relates to https://github.com/bliker/cmder/issues/280 and https://github.com/bliker/cmder/issues/218

This has been fixed since aa9cb38


⚠ Topfarmacia has degraded performance

In 82d729c, Topfarmacia (https://www.topfarmacia.it) experienced degraded performance:

HTTP code: 200
Response time: 11591 ms

Resolved: Topfarmacia performance has improved in 475c55a.


InitialValue in multiSelection

Hello,
I'm trying to put an initialValue in the multiSelection version:
initialValue: const ["SVP", "Surf", "Teste"]
And the output is:

The selections are not ok!

Hi @acamenhas
Sorry for the incovience,
I have published new version of [Dropdown_textfield](https://pub.dev/packages/dropdown_textfield) and fixed reported bug in this version ,
Please revert if any further queries.

Hi @acamenhas
Sorry for the incovience,
I have published new version of Dropdown_textfield and fixed reported bug in this version ,
Please revert if any further queries.


Fix throttled status updates by ensuring GUI updates occur on the main thread

Description
This ensures that certain GUI updates happen on the main thread. Mostly, napari is a single-threaded app, but the use of throttled in ViewerModel means that we control the frequency of a callback using many threads. Ultimately this means that some callbacks will be executed on non-main threads and will eventually try to update QWidgets which is generally unsafe.
I only handled the updates to status and help. I'm unsure if we need to handle the tooltip too.
Alternatively, we could move the connection from ViewerModel.cursor.events.position to ViewerModel._update_status_bar_from_cursor to QMainWindow or QtViewer and use qthrottled to avoid use of multiple threads. I didn't do this here because I'm unsure if any headless napari users are depending being able to manually change the cursor position and expect updates to the status/help. My guess is no, but it is a public part of the viewer model, so the approach here seemed like a better quick fix. If someone wants to create a PR with this approach, it would be interesting to compare.
Type of change

[x] Bug-fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

References
Closes #4982
Part of #4738
How has this been tested?

[x] all tests pass with my change
[x] with manual testing, I could no longer reproduce #4982

Final checklist:

[ ] My PR is the minimum possible work for the desired functionality
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If I included new strings, I have used trans. to make them localizable.
For more information see our translations guide.

I didn't do this here because I'm unsure if any headless napari users are depending being able to manually change the cursor position and expect updates to the status/help.

fair enough.  but seems incredibly unlikely :joy:
I'm happy to go with whichever you want.  but i do think it was probably the wrong place to put the connection in the first place (and it would be nice not to have to live with it! :joy:)
can come in a later PR though

I'm happy to go with whichever you want. but i do think it was probably the wrong place to put the connection in the first place (and it would be nice not to have to live with it! 😂)
can come in a later PR though

Yeah, I've also had related thoughts when it comes to async slicing. Like I said, very happy to go with an alternative PR from someone else either now or later - I just hadn't thought too much about these fields on the viewer model before today.
In general, I am drifting towards the idea that the napari model should mostly be single-threaded (at least for now). And that any multi-threading (or throttling) we add should come from the view side of the application. In this case, that means throttling from Window or QtViewer. For async slicing it could mean something like connecting slicing to widget value change events instead of model value change events.

@tlambert03 could you motivate why this PR is selected over https://github.com/napari/napari/pull/4985? Did I should close https://github.com/napari/napari/pull/4985?

Sorry, I missed that alternative, and really wanted that bug to go away.  You don't need to close it, if you think it's the better approach we can still use it

I think it is better as viewer.events.status is emitted in the main thread, but there are some questions in the discussion of this PR. and also @andy-sweet want your opinion.

@tlambert03 : yeah, I agree that #4985 is the better solution, but am fairly unfamiliar with qthrottled, so it could do with a look from you.


Semantic: turn ->var.foo into just syntax sugar of -> { var.foo }

Now, ->var.foo is just syntax sugar of -> { var.foo }.
It is PoC and an objection against @waj's comment.
/cc @hugopl

The thing is that ->foo will make a Proc out of the method that might already exist if you call foo. If you translate it as -> { foo } then there's an indirection, possibly a closure (if used like ->x.foo) and that's totally not the expected behavior. Otherwise the syntax wouldn't exist.

I see no reason why ->var.foo should closure var. It seems entirely the unintuitive behaviour to me.

What was the original motivation back in history for the creation of ->method construct?
If it was just a syntax sugar to ->{ method } then ->foo.var == ->{ foo.var } makes sense... OTOH isn't good to have multiple ways to write the same thing in a language, it just create dialects and tools to tell what construct you should use and what you should abolish.

The original motivation is to get a Proc that points directly to a function/method. That's why the AST node for that is called ProcPointer. For example you can get a pointer to a LibC function and pass it to something that expects exactly that function. If you wrap it inside another function, forming a closure, it won't work because closures can't be passed to LibC.
I don't think this PR and this direction is good.
For example in C you can do &foo to get a reference to a function. This is exactly the same, but with a different syntax.

Got it, both constructs generate a Proc without a closure, but the internal representation of ->func and ->{ func } are different, one of then having an extra pointer to hold a closure that doesn't exist.
I think that just the documentation should be updated to avoid misunderstandings.

@hugopl More or less. I don't think "closure" or "not closure" is the distinction. The closure doesn't exist if you do ->foo because doing ->{ foo() } doesn't capture any variable.
The distinction is that ->{ foo() } is a reference to a method that inside it will call foo(), while ->foo is a reference to the method foo. There's no indirection.

@asterite I'm not sure when this distinction is necessary. But I understand and accept proc pointer's motivation.

IMHO the options are:

Only allow the syntax ->something.function if something is a Class and not allow ->var.method<EMAIL_ADDRESS>and ->@@var.method. And also disallow ->foo if foo is a method... since this also creates a intermediary proc with self captured.
Use the behavior created by this PR.

I don't see the difference of capture value vs variable depending on the syntax used to add something to the language but confusion and bugs by people writing code thinking it will behave one way or another.
my 2¢

If this is causing confusion then we need more docs, not change the behaviour. As I explained in #9268 without this syntax, obtaining the same result would be more complicated. I think it's clear once you understand that the generated proc doesn't have any relationship with the variable holding the value. It's just copying the value content.
@MakeNowJust consider an example where a proc is returned from an object method, but you want to construct that proc holding just a reference to the value. Otherwise the whole object is referenced and will not be released by the GC until the proc is also unreferenced.
TBH there is no chance we can change this behaviour for this version. I'm closing this PR for now. You should open a RFC instead, and in the best case that could be changed for Crystal 2.0, but not before.


fix(astro): better default meta tags

This PR proposes to use the site logo as the default open graph image, and sets up better default meta tags for opengraph and twitter.
(please double check as opengraph display is usually tested on deployed instances, I couldn't test them locally aside from reading them in the browsers element tab)
Btw I don't think the BASE_URL env variable is documented, to have a proper absolute URL for meta tags.

I'll take a look at the code next week, but another idea came into my mind: Should we create new MetaData component that authors could use to define completely custom meta tags? https://tutorialkit.dev/guides/overriding-components/
So something like:
import tutorialkit from '@tutorialkit/astro';
import { defineConfig } from 'astro/config';

export default defineConfig({
  integrations: [
    tutorialkit({
      components: {
        MetaData: './src/components/MetaTags.astro',
      },
    }),
  ],
});

And then in ./src/components/MetaTags.astro you would get access to default meta tags, and you could decide whether to use them or not. And of course add your custom ones there too.
Improving default meta tags is important too.

Definitely, I've thought about a Metadata component but then the question is how to pass the settings from a lesson to this component
Maybe using the lesson slug as a unique id
This also joins the question of accessing lesson metadata from the content collection to generate tables of content rss feeds etc.

I have a small doubt on the update I've done in the documentation : I documented the BASE_URL variable, which is used to compute the logo, favicon or public images. Initially I thought this would the absolute URL of the application (you want absolute urls for open graph tags) but I am not sure. It seems to be a variable set by Astro and which might rather be a kind of separate directory, eg to handle a monorepo.
But I am not sure, wha'ts your opinion on this @Nemikolh ?

I still don't exactly get what BASE_URL is, but I figured that Astro.site is what I want for absolute URLs, I've pushed an update in this direction.
It's not super elegant but the SEO tags are at least correct.

Hey @eric-burel !
Do you think you could rebase this PR on top of the latest version of main? This would help reviewing this.
So you do git fetch <stackblitz-tk-remote> and then git rebase <stackblitz-tk-remote>/main. Trying this, you should only get a single conflict on packages/astro/src/default/layouts/Layout.astro.
If you want I can also do it for you, but that means that I'll force push to your branch and you'll have to reset it to the origin. Let me know what you prefer :raised_hands:

@Nemikolh done, I think I've messed up at some point and used a "merge" instead. It has fixed the diff :ok_hand:

I've also updated the logic to access the site absolute URL, I found another doc explaining that Astro.site will feed the import.meta.env.SITE env variable for consumption in JS files https://docs.astro.build/en/guides/environment-variables/
Ideally this would be set to "http://localhost:4321" in dev but I am not sure how to reliably get the right port.
Astro.url automatically has an absolute URL in development but it's not the right solution, because it's based on the current HTTP request to get a full path while we just want the website root URL so the site value.

Looking at the PR again, I would keep the previous usage of BASE_URL in place of site for resources loaded by the browser (so everything that is not OG or Twitter related) as those links can be relative and don't have to be absolute.
A big downside of forcing them to be absolute is that it makes preview deployment with cloudflare or netlify a bit more annoying as they would need a custom build.
Regarding your question on what BASE_URL is, see the docs on base.

Hi, I'll get back at the few last comments, but in the meantime I've opened a proposal for a portal system in Astro: https://github.com/withastro/roadmap/discussions/1032
This would make it easier to support any type of tag that lands into <head> without having to support a full-fledged intermediate JSON représentation that fits into the YAML frontmatter. Not that it's impossible, it's Next.js approach with the metadata object, but maybe more work than it should at framework-level for something that could be done in user-land with good old HTML.
I am thinking of alternate links for instance that I will need in AstroPatterns/NextPatterns because I translate similar content for 2 frameworks and may be punished by Google for that if I don't provide alternate links.

Looking at the PR again, I would keep the previous usage of BASE_URL in place of site for resources loaded by the browser (so everything that is not OG or Twitter related) as those links can be relative and don't have to be absolute.

BASE_URL is still there, and optionally the absolute option can be used to also append the whole website URL via Astro.site. site is only actually used when using the logo as the default ogImage, though I think having it as an option in readPublicAsset may be useful.

A big downside of forcing them to be absolute is that it makes preview deployment with cloudflare or netlify a bit more annoying as they would need a custom build.

If you don't set Astro.site you'll just come up with a warning. I think you are expected to use an env variable in astro.config to handle these cases? Eg Vercel preview URLs.
Let me know if I got something wrong!


python36Packages.influxdb: 4.0.0 -> 5.2.0

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/python3.6-influxdb/versions.
meta.description for python36Packages.influxdb is: '"Python client for InfluxDB"'.

Checks done (click to expand)

built on NixOS

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 5.2.0 with grep in /nix/store/z1kh2laa8qxlinhq0sqhxkwkmnfyhzhy-python3.6-influxdb-5.2.0

directory tree listing: https://gist.github.com/07024f0717fdfe8218d20f3dbae6a841

du listing: https://gist.github.com/3d12c0f04acd14dbed0c896f188f6bd3

Outpath report (click to expand)

Outpath difference report
16 total rebuild paths
4 package rebuilds
4 x86_64-linux rebuilds
4 i686-linux rebuilds
4 x86_64-darwin rebuilds
4 aarch64-linux rebuilds
First ten rebuilds by attrpath
python27Packages.influxdb
python27Packages.influxgraph
python36Packages.influxdb
python36Packages.influxgraph

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/z1kh2laa8qxlinhq0sqhxkwkmnfyhzhy-python3.6-influxdb-5.2.0 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A python36Packages.influxdb https://github.com/r-ryantm/nixpkgs/archive/dae80d35fdff860f2c88c2efafd5f251bf1fa228.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/z1kh2laa8qxlinhq0sqhxkwkmnfyhzhy-python3.6-influxdb-5.2.0
ls -la /nix/store/z1kh2laa8qxlinhq0sqhxkwkmnfyhzhy-python3.6-influxdb-5.2.0/bin

Success on x86_64-linux (full log)
Attempted: python36Packages.influxdb
Partial log (click to expand)
Installing collected packages: influxdb
Successfully installed influxdb-5.2.0
/build/influxdb-5.2.0
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/z1kh2laa8qxlinhq0sqhxkwkmnfyhzhy-python3.6-influxdb-5.2.0
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/z1kh2laa8qxlinhq0sqhxkwkmnfyhzhy-python3.6-influxdb-5.2.0/lib
patching script interpreter paths in /nix/store/z1kh2laa8qxlinhq0sqhxkwkmnfyhzhy-python3.6-influxdb-5.2.0
checking for references to /build in /nix/store/z1kh2laa8qxlinhq0sqhxkwkmnfyhzhy-python3.6-influxdb-5.2.0...
/nix/store/z1kh2laa8qxlinhq0sqhxkwkmnfyhzhy-python3.6-influxdb-5.2.0


linux-firmware: Add firmware for Intel wifi chipset on NUC 10th gen

Changelog-entry: Add firmware for Intel wifi chipset on NUC 10th gen
Signed-off-by: Florin Sarbu<EMAIL_ADDRESS>

@balena-ci retest


add refrence

not able to add in refrence c# error "Please make sure that the file is accessible and that it is a valid assembly or COM component."

Yo do not necesite add exe or dll file.
Just add WebPWrapper.cs to your proyect files.


chore(rpc): EthTransactions default trait methods using pending block

Moves EthTransactions impl on EthApi, that uses pending block, to default trait methods

blocked by https://github.com/paradigmxyz/reth/pull/8640


pylint cleanup

It turns out that our pre-commit config was actually not running the pylint tests.  I fixed that here.  Here are the major changes in this PR:

Change mentioned above
Most changes are import-level changes (deleting unused imports or reordering them).
A couple of changes suggested by pylint, along with disabling some checks on a line/function basis so we can come back to them.  The code works now, and this PR shouldn't break anything.
PR #508 introduced cyclic imports in lib/condor.py that both the author and reviewer missed.  Unit and integration tests passed, so the fact that pylint wasn't working before meant that this was missed when development on that PR was being done.  We need to decide whether to fix that issue in this PR, or open a new issue for that.  If the latter, we should definitely have it fixed before the next release.

Before anyone reviews, I'll check to see why the CI tests failed and try to resolve those.

So unfortunately, according to the docs here:  https://pylint.pycqa.org/en/latest/user_guide/installation/pre-commit-integration.html, pylint has to be run locally (not inside an isolated environment as pre-commit would normally do).  As such, I've made it so that the pylint tests work if a developer has installed pre-commit, but are skipped for CI, which they already were.  I think as a separate issue, I'll get a github actions container spun up where we can, among unit tests, run pylint on the code base.


torch shape mismatch while training the language model.

hi,
While training the language model , Iam facing the torch shape mismatch error:
I have attached the screenshot for the same.
please help

You need to use the gdb13ScaffoldCharTokenizer

Hi,
I changed the Tokeniser but now facing error in the encoding

is this error  because of dataset format as the error mention it is not able to tokenise '\t'.
iam attaching the screenshot of dataset

Thank you


Fix add error log to rest response

Description
Empty error logs cause extra logging.
https://trello.com/c/Gz3CNEEL
Summary of changes:

Added check for empty error log messages.

Closing due to extra commits.


Figure out way to know which wallets support setting feeCurrency

Wallets designed for Celo like valora, CeloWallet,  offer the capability to add an additional field to tx for feeCurrency. wallets which support only ethereum style tx like metamask do not.
Right now use-ck offers functions to change fee currency regardless of wallet. we should be more discerning.

Hearing that when using wallet connect v1 the fee currency is param is not sent to wallet anyway.

This was fixed by #168


Minor fixes in Czech translations

Fixes #94

Thank you! We'll get these live on our next production deploy.


Develop

Pull Request Test Coverage Report for Build 561730747

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

Totals

Change from base Build 171:
0.0%

Covered Lines:
271

Relevant Lines:
429

💛  - Coveralls


Compilation warning

src/src/gen-noise.cpp||In function ‘FN_DECIMAL FastAbs(FN_DECIMAL)’:|
src/src/gen-noise.cpp|187|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘double’ may alter its value [-Wfloat-conversion]|
src/src/gen-noise.cpp||In member function ‘void FastNoise::SetSeed(int)’:|
src/src/gen-noise.cpp|204|warning: conversion to ‘unsigned char’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|212|warning: conversion to ‘unsigned char’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp||In function ‘FN_DECIMAL ValCoord2D(int, int, int)’:|
src/src/gen-noise.cpp|281|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp||In function ‘FN_DECIMAL ValCoord3D(int, int, int, int)’:|
src/src/gen-noise.cpp|290|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp||In function ‘FN_DECIMAL ValCoord4D(int, int, int, int, int)’:|
src/src/gen-noise.cpp|300|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp||In member function ‘FN_DECIMAL FastNoise::GradCoord4D(unsigned char, int, int, int, int, FN_DECIMAL, FN_DECIMAL, FN_DECIMAL, FN_DECIMAL) const’:|
src/src/gen-noise.cpp|326|warning: conversion to ‘unsigned char’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp||In member function ‘FN_DECIMAL FastNoise::SingleSimplex(unsigned char, FN_DECIMAL, FN_DECIMAL, FN_DECIMAL) const’:|
src/src/gen-noise.cpp|1083|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1084|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1085|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1086|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1126|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1127|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1128|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1129|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1130|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1131|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp||In member function ‘FN_DECIMAL FastNoise::SingleSimplex(unsigned char, FN_DECIMAL, FN_DECIMAL) const’:|
src/src/gen-noise.cpp|1281|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1282|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1283|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1337|warning: conversion to ‘FN_DECIMAL {aka float}’ alters ‘double’ constant value [-Wfloat-conversion]|
src/src/gen-noise.cpp|1338|warning: conversion to ‘FN_DECIMAL {aka float}’ alters ‘double’ constant value [-Wfloat-conversion]|
src/src/gen-noise.cpp||In member function ‘FN_DECIMAL FastNoise::SingleSimplex(unsigned char, FN_DECIMAL, FN_DECIMAL, FN_DECIMAL, FN_DECIMAL) const’:|
src/src/gen-noise.cpp|1348|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1349|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1350|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1351|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1352|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1385|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1386|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1387|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1388|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1389|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1390|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1391|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1392|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1393|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1394|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1395|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1396|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp||In member function ‘FN_DECIMAL FastNoise::SingleCellular(FN_DECIMAL, FN_DECIMAL, FN_DECIMAL) const’:|
src/src/gen-noise.cpp|1712|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1713|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1714|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1738|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1739|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1740|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1764|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1765|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1766|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1795|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1795|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1795|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp||In member function ‘FN_DECIMAL FastNoise::SingleCellular2Edge(FN_DECIMAL, FN_DECIMAL, FN_DECIMAL) const’:|
src/src/gen-noise.cpp|1823|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1824|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1825|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1830|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘double’ may alter its value [-Wfloat-conversion]|
src/src/gen-noise.cpp|1831|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘double’ may alter its value [-Wfloat-conversion]|
src/src/gen-noise.cpp|1845|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1846|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1847|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1852|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘double’ may alter its value [-Wfloat-conversion]|
src/src/gen-noise.cpp|1853|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘double’ may alter its value [-Wfloat-conversion]|
src/src/gen-noise.cpp|1867|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1868|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1869|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1874|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘double’ may alter its value [-Wfloat-conversion]|
src/src/gen-noise.cpp|1875|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘double’ may alter its value [-Wfloat-conversion]|
src/src/gen-noise.cpp||In member function ‘FN_DECIMAL FastNoise::SingleCellular(FN_DECIMAL, FN_DECIMAL) const’:|
src/src/gen-noise.cpp|1935|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1936|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1956|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1957|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1977|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|1978|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|2003|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|2003|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp||In member function ‘FN_DECIMAL FastNoise::SingleCellular2Edge(FN_DECIMAL, FN_DECIMAL) const’:|
src/src/gen-noise.cpp|2029|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|2030|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|2035|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘double’ may alter its value [-Wfloat-conversion]|
src/src/gen-noise.cpp|2036|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘double’ may alter its value [-Wfloat-conversion]|
src/src/gen-noise.cpp|2047|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|2048|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|2053|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘double’ may alter its value [-Wfloat-conversion]|
src/src/gen-noise.cpp|2054|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘double’ may alter its value [-Wfloat-conversion]|
src/src/gen-noise.cpp|2065|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|2066|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘int’ may alter its value [-Wconversion]|
src/src/gen-noise.cpp|2071|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘double’ may alter its value [-Wfloat-conversion]|
src/src/gen-noise.cpp|2072|warning: conversion to ‘FN_DECIMAL {aka float}’ from ‘double’ may alter its value [-Wfloat-conversion]|
src/src/gen-noise.cpp|186|warning: ‘int FastAbs(int)’ defined but not used [-Wunused-function]|

@Auburns no comments? Why invalid?

You gave no info about the issue, compiler version, compiler settings, are you using doubles/floats?

fair enough. I just tested this on another system and got even more warnings, possibly because it has newer gcc version (9.2.0), Arch Linux.
Here's part of the Makefile:
CC      = g++
PROF    = -O3
C_FLAGS = -std=c++14 -Wall -Wextra -pedantic-errors -Wconversion -Wno-unused-parameter -fmax-errors=5 $(PROF)

Getting a lot of stuff like this:
warning: conversion from ‘int’ to ‘unsigned char’ may change value [-Wconversion]

warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
*reinterpret_cast<int*>(&x) ^ (*reinterpret_cast<int*>(&x) >> 16)

In member function ‘FN_DECIMAL FastNoise::GetNoise(FN_DECIMAL, FN_DECIMAL, FN_DECIMAL) const’:
warning: this statement may fall through [-Wimplicit-fallthrough=]
switch (m_fractalType)

In member function ‘FN_DECIMAL FastNoise::GetValue(FN_DECIMAL, FN_DECIMAL, FN_DECIMAL) const’:
warning: ‘zs’ may be used uninitialized in this function [-Wmaybe-uninitialized]
static FN_DECIMAL Lerp(FN_DECIMAL a, FN_DECIMAL b, FN_DECIMAL t) { return a + t * (b - a); }


how to enable "enableBlockReuse" ?

"enableBlockReuse" in batchmanager config is false in default, how to enable enableBlockReuse True ?

If you use triton backend, it is controlled by enable_kv_cache_reuse.

I built the qwen model engine using command as fellow:

The following error occurred using the above method：

This feature is only exposed in gpt model now https://github.com/NVIDIA/TensorRT-LLM/blob/main/examples/gpt/build.py#L362-L367. For other models, it requires works to add this config to enable it.

Hi @yunzhongyan0 do u still have further issue or question now? If not, we'll close it soon.


Allow FreeDraw shapes to end close to their start

✓ New initially-failing tests
✓ Aaaand they pass
✓ Aaaaand one of them showed me that I was forgetting to include the ending position in the distance calculation so that's fixed
Was a pretty nice experience overall.
Fixes #84.
Before

After

merged bc it's a good fix for the next release. can (maybe?) rename things whenever


Support customized .terraformrc , credentials.tfrc.json and credentials-helper for private terraform registry

Sometimes users want to use customized .terraformrc , credentials.tfrc.json and credentials helper files. Here's one use case:
Use self-hosted JFrog Artifactory to proxy and cache AWS provider resource. These files are used to point to the self-hosted services and provide credentials. This is related to https://github.com/kubevela/terraform-controller/issues/350
The schematic terraform cue must support passing the required configuration for mounting the above files by passing it on to the terraform controller if present.

Fixed in #5550


Add jsdom specific tests for VideoControls

Sets up test files named *.jsdom.test.js to only be run in the jsdom environment. Regular .test.js files will still be run in both jsdom and node environments.

Deploy preview for gestalt ready!
Built with commit<PHONE_NUMBER>58470ade795e87f0400d5f3cf384ad
https://deploy-preview-205--gestalt.netlify.com

Deploy preview for gestalt ready!
Built with commit cea020310262413c4c58701795d8f274d84ced3d
https://deploy-preview-205--gestalt.netlify.com

Codecov Report

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

@@            Coverage Diff             @@
##           master     #205      +/-   ##
==========================================
+ Coverage   61.67%   62.56%   +0.88%
==========================================
  Files          40       40
  Lines         788      788
  Branches      183      183
==========================================
+ Hits          486      493       +7
+ Misses        214      209       -5
+ Partials       88       86       -2

Impacted Files
Coverage Δ

packages/gestalt/src/Touchable/Touchable.js
53.84% <0%> (+7.69%)
:arrow_up:

packages/gestalt/src/Video/VideoControls.js
86.36% <0%> (+27.27%)
: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 89b4933...cea0203. Read the comment docs.


Parallel tests

Proposed Changes

Parallelize test cases.

Trying to keep this pattern so when we inevitably copy/paste them they're parallel
Should run faster

/lgtm

/lgtm


Added (most) RadioPlayer sites

Added support back for the RadioPlayer sites... managed to get both radio player layouts working. Just need to get the BBC style ones working (as they have the ability to scrub back along the live audio stream)

Just added my own station as a connector... I own Delta Radio UK...

You can run grunt eslint to fix ESLint errors.

@alexesprit cheers :)

Added in master branch.


getting error:  cannot find module providing package archive/zip

Hi,
I am running windows 7 Enterprise, 64-bit OS.
I tried "go get github.com/rakyll/statik"
but got this error:
build github.com/rakyll/statik: cannot load archive/zip: cannot find module providing package archive/zip
What i am missing here?

You need to include the generated package. See https://github.com/rakyll/statik/issues/32


fix yaml dump

Hi Marvin  @pmrv  ,
I've checked the yaml fix you suggested and it works. Thanks a lot for the help. Here is the fix.
Best,
Jing

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
188

Relevant Lines:
5668

💛  - Coveralls


Make the LIDAR Lite driver more resilient to device failures

This change adds:

retries for nacks
reset (thanks to Andrew Tridgell)
stats to help observe nack behavior, this may be trimmed down
displaying the hardware and firmware version numbers

Two bench tests of about 1 hour each survived with the lenses covered. The second test failed at the one hour mark.
state @ 20014440
Hardware version: 1
Firmware version: 1
ll40ls_read: 49830 events, 0 overruns, 69471654us elapsed, 1394us avg, min 574us max 36954us 3199.614us rms
ll40ls_measure_nacks: 0 events
ll40ls_collect_nacks: 65 events
ll40ls_collect_nacks_1: 15 events
ll40ls_collect_nacks_2: 15 events
ll40ls_collect_nacks_3: 14 events
ll40ls_collect_nacks_4: 14 events
ll40ls_collect_nacks_5: 7 events
ll40ls_collect_nacks_6: 0 events
ll40ls_resets: 0 events
ll40ls_reset_nacks: 15 events
ll40ls_comms_errors: 0 events
ll40ls_buffer_overflows: 49829 events
poll interval:  17 ticks
report queue    2/80 (1/0 @ 20016370)
distance: 73cm (0x0049)

@tridge I'm getting up to about an hour with the lenses covered. The first test I stopped after over an hour, the second failed just before the one hour mark.
Can you give it a try and see if it improves things? Are you using the prototype or do you have a more recent board? The ones I'm testing are the Kickstarter ones from late Dec. 2014.

@tridge not only try without an LED connected but with the LIDAR connected directly to the Pixhawk I2C port.

Closing this as Andrew will have a better solution in the pipeline and has been working closely with PulsedLight. The best workaround for now is probably just not to put anything else on the I2C bus.


error: blocks support disabled

Brief Description
Hi all, I'm having trouble in generating C# bindings for macOS Endpoint Security API. Below you have more details and the full error message, but I would kindly ask you the following:

Is what I'm trying to achieve doable with CppSharp or not?
Would it generate meaningful bindings if I manage to enable blocks support?
If not, could I manually write C# p/invoke bindings for that single problematic function expecting that block type as argument and let CppSharp deal with the remaining part of those headers?

Cheers,
LuKe
OS: macOS 10.15.7
SDK: .NET Core 3.1
Used headers

EndpointSecurity/EndpointSecurity.h, which is a wrapper for:

EndpointSecurity/ESTypes.h
EndpointSecurity/ESMessage.h
EndpointSecurity/ESClient.h

located at: /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/
The problematic line is at ESClient.h(187,15) and contains this block type definition: https://developer.apple.com/documentation/endpointsecurity/es_handler_block_t
Used settings
Nothing special so far, same as in CppSharp's Getting Started page. I built nupkgs using dotnet pack command, then referenced CppSharp.Generator from inside a new project and implemented Setup() adding just EndpointSecurity/EndpointSecurity.h as header, which is found as expected.
I had to set DYLD_LIBRARY_PATH env variable in project "Run" settings to /full_path_to/CppSharp/bin/Release_x64 to let it load libCppSharp**.dylib, and I copied lib folder from inside the same folder into my project bin/Release output folder.
Stack trace
Error parsing 'EndpointSecurity/EndpointSecurity.h'
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/EndpointSecurity/ESClient.h(187,15): error: blocks support disabled - compile with -fblocks or pick a deployment target that supports them
CppSharp has encountered an error while parsing code.

Try adding the parserOptions.TargetTriple = "x86_64-apple-darwin" as an option to setup the parser in Mac mode.

Try adding the parserOptions.TargetTriple = "x86_64-apple-darwin" as an option to setup the parser in Mac mode.

Hi, @tritao
adding parserOptions.TargetTriple = "x86_64-apple-darwin" I get an additional error about a missing def:
Error parsing 'EndpointSecurity/EndpointSecurity.h'
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/EndpointSecurity/ESMessage.h(19,3): This header requires __DARWIN_64_BIT_INO_T
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/EndpointSecurity/ESClient.h(187,15): error: blocks support disabled - compile with -fblocks or pick a deployment target that supports them
CppSharp has encountered an error while parsing code.

Hi, @tritao
adding parserOptions.TargetTriple = "x86_64-apple-darwin" I get an additional error about a missing def:
Error parsing 'EndpointSecurity/EndpointSecurity.h'
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/EndpointSecurity/ESMessage.h(19,3): This header requires __DARWIN_64_BIT_INO_T
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/EndpointSecurity/ESClient.h(187,15): error: blocks support disabled - compile with -fblocks or pick a deployment target that supports them
CppSharp has encountered an error while parsing code.

It's possible that header is not meant to be included standalone, or only after some other foundation header has been included.
Anyway you can try to set Defines option with it, and to pass -fblocks explicitly as well to see if that gets you going.
Btw what happens if you try calling Clang on a file with just that include? Does it work?

It's possible that header is not meant to be included standalone, or only after some other foundation header has been included.
Anyway you can try to set Defines option with it, and to pass -fblocks explicitly as well to see if that gets you going.
Btw what happens if you try calling Clang on a file with just that include? Does it work?

or only after some other foundation header has been included

I see a Foundation/Foundation.h being included in a sample app using that API, however adding it to module.Headers made no difference, it seems to get parsed fine but that flag is still missing

Btw what happens if you try calling Clang on a file with just that include? Does it work?

Yes, running clang (which is /urs/bin/clang) onto a minimal:
#include <EndpointSecurity/EndpointSecurity.h>

int main() {}

just works, no Foundation.h included, no -fblocks passed

Anyway you can try to set Defines option with it,

I wonder why I don't get the  missing __DARWIN_64_BIT_INO_T error when TargetTriple is not being set by me in Setup()
I added a driver.ParserOptions.AddDefines("__DARWI_64_BIT_INO_T") and it now goes on

and to pass -fblocks explicitly as well to see if that gets you going.

adding driver.ParserOptions.AddArguments("-fblocks") made the blocks support error go away
Now I see no more errors, it says Processing code... , Generating code... and Generated Std.cs which is quite empty (ie. just five using statements

or only after some other foundation header has been included

I see a Foundation/Foundation.h being included in a sample app using that API, however adding it to module.Headers made no difference, it seems to get parsed fine but that flag is still missing

Btw what happens if you try calling Clang on a file with just that include? Does it work?

Yes, running clang (which is /urs/bin/clang) onto a minimal:
#include <EndpointSecurity/EndpointSecurity.h>

int main() {}

just works, no Foundation.h included, no -fblocks passed

Anyway you can try to set Defines option with it,

I wonder why I don't get the  missing __DARWIN_64_BIT_INO_T error when TargetTriple is not being set by me in Setup()
I added a driver.ParserOptions.AddDefines("__DARWI_64_BIT_INO_T") and it now goes on

and to pass -fblocks explicitly as well to see if that gets you going.

adding driver.ParserOptions.AddArguments("-fblocks") made the blocks support error go away
Now I see no more errors, it says Processing code... , Generating code... and Generated Std.cs which is quite empty (ie. just five using statements

Right now we skip almost all system declarations by default (with an exception for C++ standard library types) for performance reasons. This could be the reason why its not generating anything. We should add an option to disable this for these cases of binding system headers.
You could try to change the Parser::IsSupported methods to just return true: https://github.com/mono/CppSharp/blob/master/src/CppParser/Parser.cpp#L899
Another easier alternative that might work is to copy the headers you need to a local folder (outside system include folders) and try to bind those instead.

Right now we skip almost all system declarations by default (with an exception for C++ standard library types) for performance reasons. This could be the reason why its not generating anything. We should add an option to disable this for these cases of binding system headers.
You could try to change the Parser::IsSupported methods to just return true: https://github.com/mono/CppSharp/blob/master/src/CppParser/Parser.cpp#L899
Another easier alternative that might work is to copy the headers you need to a local folder (outside system include folders) and try to bind those instead.

Moving those headers to the desktop worked, it's now generating the expected code. Of course it crashes as soon as it finds the block_t def. I managed to get it complete by commenting out that block_t definition and the only function using it.
Now I need to figure out how to translate that block_t into C#, it could be similar to a delegate.

Moving those headers to the desktop worked, it's now generating the expected code. Of course it crashes as soon as it finds the block_t def. I managed to get it complete by commenting out that block_t definition and the only function using it.
Now I need to figure out how to translate that block_t into C#, it could be similar to a delegate.


in is a reserved word.

"in" is a reserved word in javascript. Wouldn't it be cleaner to rename the in()-method into something like inList() ?

Please re-open this issue.  Browsers like IE 8 do not like 'Validator.prototype.in'  You might be able to change this to Validator.prototype['in']
Thanks!

Confirmed.  You can change Validator.prototype.in to Validator.prototype['in'].  Browsers with more strict (or maybe just plain dumb) JavaScript engines will no longer complain, and actually, this is much better anyway since 'in' is technically a reserved word.


Remove fix for diagnostic CLI bug upstream

Once typst/typst#1937 has been resolved we can remove the fix done in #44.

Seems fixed by https://github.com/typst/typst/commit/afc95ed19db8ded044d3b9b916fa0194cb882d0b (I tried for my usecase), but it's not in a release yet.


Search filtering restricts results too much

Describe the bug
When doing a search for different objects in the ATT&CK extension, the filter requires all of an object's types to be active before that object appears in the results.
For instance, if we search "Panda" with the "Enterprise" and "Group" filters on, we can see various groups in the results.

But if either one of those filters are switched off, we do not get any results, even though the objects clearly have both tags.

Another example of this behaviour would be when searching for "phishing". A lot of results have the "Enterprise" and "Techniques" type assigned to them. If either "enterprise" or "techniques" are turned off in the filters, there will be no results shown.
If this is expected behaviour, implementing the option to have a less restrictive filter might be a good idea.

To Reproduce
Provide detailed steps to reproduce this outcome.

Select type of objects you want to filter in extension
If a result has other types assigned to it (besides the selected ones), they will not show up.

Platform:

OS: Windows
Browser: Firefox
Browser Version: 111.0.1 (64-bit)

If you want to express positive or negative feedback on a bug report, please leave a Thumbs Up or Thumbs Down so that we can keep track of demand. Please do not reply just to say +1 or -1. Thanks!

This is the expected behavior: the filters are AND conditions. For example, if a user is working on an enterprise security project, they can set up their filters to only see Enterprise results. I think this makes sense for the default behavior.
There is an extended search syntax (similar to Solr / Lucene syntax) to search by object type, domain, etc., and those are OR conditions:

If you scroll down a bit from that search, you'll see some results that are Group but not ICS.

I may be misunderstanding, but if the filters are AND conditions, wouldn't selecting all of them (as is default) block any results from showing?
The current behaviour seems to be (Enterprise OR ICS OR mobile) AND (other option OR other options OR etc) as highlighted by the below screenshot

Since there seems to be a distinction made between the enterprise/ICS/mobile filters and the rest of the filter types, would highlighting this difference (maybe through a visual separator) be a possible idea?

The current behaviour seems to be (Enterprise OR ICS OR mobile) AND (other option OR other options OR etc) as highlighted by the below screenshot

Yes -- this is correct (I oversimplified in my previous reply)

would highlighting this difference (maybe through a visual separator) be a possible idea?

Yes, this is a good idea! We will work this into an upcoming release.

I fixed this in #50. This is what it looks like:


link to twitchsaver in README.md

twitchsaver is a webpage that serves up a 3 x 2 grid of cathode tube TVs showing twitch.tv's featured streams.
More info: https://github.com/ameswarb/twitchsaver

It's probably worth noting that no NSFW content should appear on the streams. Twitch requires streamers to flag their streams as NSFW if applicable. NSFW flagged streams require the user to click to confirm that they want to view the content, which is obviously a problem for my use case. Because of this, my project only lists SFW streams:
for (var i = 0; i < channels.length; i++) {
    if (output.length < 6 && !channels[i].stream.channel.mature) {
      output.push(channels[i].stream.channel.name);
    }
  }

I'm going to go ahead and close this PR for now. Twitch's HTML5 player has some wonky issues which leads to my page being unreliable:

It will sometimes randomly request non-https streams, which breaks the https version because of mixed content. (necessary to support https as plugins like https everywhere try to force gh-pages sites onto https)
Twitch will sometimes decide that it wants a token for its HTML5 streams. I don't think they're targeting their HTML5 player for public consumption yet, it's more for mobile devices. Apparently from what I've read, the token generation happens 100%  in clientside javascript though, so I should be able to work around this.

Both issues are fixable, but I don't think my page is ready for primetime yet.


encrypt should smartly find a DOTENV_PRIVATE_KEY and generate a new public key from it

currently it generates a brand new private/public key pair and appends the new private key to the .env.keys file

https://github.com/dotenvx/dotenvx/pull/298/files#diff-a1b47ff9598c568501578aa14040ac895b787a9a5d11133162320e5f09476e01R3
generates same publicKey. they are deterministic.


Add ARM support to libtock_runtime.

This support includes:

ARM entry point assembly.
Changes to start_prototype so it works with the new RawSyscalls and on ARM.
A new toolchain specification in extern_asm.rs. I renamed "target" to "prefix" because on ARM the tool prefix doesn't match the target triple.
An ARM implementation of RawSyscalls on TockSyscalls.

This PR is ready to merge.

bors r+


Feature issue144

I think #144 is asking for hover on tootlip which can be implemented but somewat complex for the moment, you can check the example in Theme and delay
As for click trigger tooltip, it has already been implemented by custom event


Stable versions are pre-release versions as per semver specification

What happened?
We are versioning our stable artifacts in the following format: x.y.z-up.n. However, as per semver specification, these versions considered to be pre-release.
This is fine as long as we consume uxp packages via up cli, however, causing problems when they need to be consumed by external tooling.
We've been observing this problem with helm cli, however, up cli being our primary installation method and helm having --devel option as a workaround, this considered as a low priority issue so far.
Now, I am having a similar problem while working on publishing UXP to Rancher catalog. UXP charts with the current version format not visible in the UI unless the user intentionally enables "Include Prerelease Versions" option under preferences.
How can we reproduce it?
$ helm repo add upbound-stable https://charts.upbound.io/stable && helm repo update

$ helm search repo -l upbound-stable/universal-crossplane
No results found

$ helm search repo -l upbound-stable/universal-crossplane --devel
NAME                               	CHART VERSION	APP VERSION	DESCRIPTION
upbound-stable/universal-crossplane	1.2.1-up.3   	1.2.1-up.3 	Upbound Universal Crossplane allows you to conn...
upbound-stable/universal-crossplane	1.2.1-up.2   	1.2.1-up.2 	Upbound Universal Crossplane allows you to conn...
upbound-stable/universal-crossplane	1.2.1-up.1   	1.2.1-up.1 	Upbound Universal Crossplane allows you to conn...

What environment did it happen in?
Universal Crossplane version: 1.2.1-up.3
Kubernetes version (kubectl version): N/A

I see the following options here:
a. Keep current versioning and we are good with up cli as the main installation tool. Find workarounds for other deployment methods:

Document helm to use with --devel
Patch version in rancher marketplace, e.g. 1.2.1-up.3 --> 1.2.1003 (assuming documenting to enable "Include Prerelease Versions" option to use UXP is a no go)

b. Change our versioning scheme with the constraint of first part will match full upstream crossplane version, e.g. x.y.z-up.n -> x.y.z00n
c. Change our versioning scheme with the constraint of only major.minor will match upstream crossplane version, patch is independent, e.g. x.y.z-up.n -> x.y.n

@turkenh I don't think (c) is an option as this would be very difficult to track which patch version of Crossplane it corresponded to and would also make our requirement that upgrading from Crossplane to UXP have the same major / minor / patch require some sort of external mapping (unless we amended that rule to only require matching major / minor). For (b), it sounds unconventional, but would be valid semver and should allow us to validate versions (i.e. break on 00). Do you know if there is any way to show pre-release versions by default in the rancher marketplace?

Do you know if there is any way to show pre-release versions by default in the rancher marketplace?

It is a user preference that is off by default. I don't think (couldn't find) a way to control this at chart level.

Can we open an issue in the related Rancher repo about this? Semver is not strict as it uses MAY statement, so there might be some room there.
If we think there is no way around and we absolutely want customers to see the entry without turning pre-release option on, then option (b) sounds like the best among options that gives consistency and less confusion.

Can we open an issue in the related Rancher repo about this? Semver is not strict as it uses MAY statement, so there might be some room there.

I believe the MAY there is to tell semver may (or may not) have a prerelease part. It is not about that part may not be considered as a prerelease.
Also, this is how existing, semver libraries being implemented and the rancher dashboard just rely on an existing library to parse semver. So, I don't think we can ask to make such an exception there.
From Rancher, I can only think of asking for a chart-level option for this but this would mean we are choosing option a, which is finding workarounds to this more general problem (and we already have a workaround as mentioned). This could still be an option that we might want to choose.

We have been living with this for more than 3 years without too much confusion and enough workarounds.
Also, given there is no easy and nice solution, closing this issue was not planned.


Verify fails on records downloaded from LR

This is the issue that Steve L. reported on the dev list.
"Both published fine, but when I downloaded the document and ran the verify code, it failed.  I'm just wondering if anyone else is having trouble verifying signatures."
http://groups.google.com/group/learningreg-dev/browse_thread/thread/b7f7e9a6dc535db2

Thanks Jim.  The same result occurs if I specific the path to gpg.exe, which should have been in the path anyhow:

type obtained.json | python -m LRSignature.cmd verify --gnupghome "C:/Users/548580/AppData/Roaming/gnupg/" --gpgbin ""C:\Program Files (x86)\GNU\GnuPG\pub\gpg.exe""

{"results": [{"doc_ID": "aa0e6a1277834f3ea9298de0f585bc20", "verified": null}]}
Here is the source envelope:
'''{
"TOS": {
"submission_TOS": "http://www.learningregistry.org/tos/cc0/v0-5/"
},
"active": true,
"doc_type": "resource_data",
"doc_version": "0.23.0",
"identity": {
"curator": "Damon Regan",
"owner": "Khan Academy",
"submitter": "Damon Regan",
"signer": "Damon Regan",
"submitter_type": "user"
},
"keys": [
"khan academy",
"math",
"subtracting fractions"
],
"resource_locator": "http://www.khanacademy.org/video/subtracting--fractions?topic=developmental-math-1",
"resource_data_type": "metadata",
"payload_placement": "inline",
"payload_schema_locator": "http://ns.nsdl.org/nsdl_dc_v1.02/ http://ns.nsdl.org/schemas/nsdl_dc/nsdl_dc_v1.02.xsd",
"payload_schema": [
"nsdl_dc"
],
"resource_data": "<nsdl_dc:nsdl_dc xmlns:nsdl_dc="http://ns.nsdl.org/nsdl_dc_v1.02/\" xmlns:dc="http://purl.org/dc/elements/1.1/\" xmlns:dct="http://purl.org/dc/terms/\" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance\" xmlns="http://www.openarchives.org/OAI/2.0/\" schemaVersion="1.02.000" xsi:schemaLocation="http://ns.nsdl.org/nsdl_dc_v1.02/ http://ns.nsdl.org/schemas/nsdl_dc/nsdl_dc_v1.02.xsd\">  dc:titleSubtracting Fractions</dc:title><dc:identifier xsi:type="dct:URI">http://www.khanacademy.org/video/subtracting--fractions?topic=developmental-math-1</dc:identifier>dc:subjectDevelopmental Math</dc:subject><dct:educationLevel xsi:type="nsdl_dc:NSDLEdLevel">Informal Education</dct:educationLevel>dc:descriptionA 2 minute video from Salman Khan explaining how to subtract fractions.</dc:description><dc:type xsi:type="nsdl_dc:NSDLType"">Lecture/Presentation</dc:type><dc:type xsi:type="nsdl_dc:NSDLType">Tutorial</dc:type><dc:type xsi:type="nsdl_dc:NSDLType">Audio/Visual</dc:type>dc:typeVideo</dc:type>dc:rightshttp://www.khanacademy.org/about/tos</dc:rights><dc:format xsi:type="dct:IMT">video/x-flv</dc:format><dc:language xsi:type="dct:RFC3066">en</dc:language>dc:publisherKhan Academy</dc:publisher></nsdl_dc:nsdl_dc>"
}'''
Here is the signed envelope:
{"documents": [{"doc_type": "resource_data", "digital_signature": {"key_location": ["http://www.damonregan.com/gpg/public-key.txt"], "key_owner": "Damon Regan<EMAIL_ADDRESS>"signing_method": "LR-PGP.1.0", "signature": "-----BEGIN PGP SIGNED MESSAGE-----\r\nHash: SHA1\r\n\r\n592858196e94f6b6b03d29f3ed566a95018275fdaa311133c8bd24e48657c960\r\n-----BEGIN PGP SIGNATURE-----\r\nVersion: GnuPG v2.0.17 (MingW32)\r\n\r\niQEcBAEBAgAGBQJPOT9rAAoJEHSrmNzJkYrV3CcIAK77voixvNpgPm2a/vQdj7zF\r\nZRUeqez3nqPVw09/yNB/ZKhsx7DrjShESiuqZU+JO4IaRa2CdXnp+rVrLBMad2Kg\r\n82EYFGbVi0TG4OLpMSd/rYvx3QtGrfyL5cyOHElKAeIlYDb6cbTO/yP7CQ9gRo36\r\nEQndAQceGeg64wtQGrcFHIk8L1rEQ8k2KWezBEh+gLQCckiOSjn2DUIPDBwwT5vA\r\nwyjT02pH9nZTsR3zbBxw9Oii7vNZDYPj1iok4cFR2/uZhpON3DfQCKAhiKvhvErI\r\nDStEQb4Ou9qn0Du+gIe8K9SeRXYRTEq3SDcuMAug0F8I5b9x3RfVHR33fLnr0Yw=\r\n=okni\r\n-----END PGP SIGNATURE-----\r\n"}, "resource_data": "<nsdl_dc:nsdl_dc xmlns:nsdl_dc="http://ns.nsdl.org/nsdl_dc_v1.02/\" xmlns:dc="http://purl.org/dc/elements/1.1/\" xmlns:dct="http://purl.org/dc/terms/\" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance\" xmlns="http://www.openarchives.org/OAI/2.0/\" schemaVersion="1.02.000" xsi:schemaLocation="http://ns.nsdl.org/nsdl_dc_v1.02/ http://ns.nsdl.org/schemas/nsdl_dc/nsdl_dc_v1.02.xsd\">  dc:titleSubtracting Fractions</dc:title><dc:identifier xsi:type="dct:URI">http://www.khanacademy.org/video/subtracting--fractions?topic=developmental-math-1</dc:identifier>dc:subjectDevelopmental Math</dc:subject><dct:educationLevel xsi:type="nsdl_dc:NSDLEdLevel">Informal Education</dct:educationLevel>dc:descriptionA 2 minute video from Salman Khan explaining how to subtract fractions.</dc:description><dc:type xsi:type="nsdl_dc:NSDLType"">Lecture/Presentation</dc:type><dc:type xsi:type="nsdl_dc:NSDLType">Tutorial</dc:type><dc:type xsi:type="nsdl_dc:NSDLType">Audio/Visual</dc:type>dc:typeVideo</dc:type>dc:rightshttp://www.khanacademy.org/about/tos</dc:rights><dc:format xsi:type="dct:IMT">video/x-flv</dc:format><dc:language xsi:type="dct:RFC3066">en</dc:language>dc:publisherKhan Academy</dc:publisher></nsdl_dc:nsdl_dc>", "keys": ["khan academy", "math", "subtracting fractions", "lr-test-data"], "TOS": {"submission_TOS": "http://www.learningregistry.org/tos/cc0/v0-5/"}, "resource_data_type": "metadata", "payload_schema_locator": "http://ns.nsdl.org/nsdl_dc_v1.02/ http://ns.nsdl.org/schemas/nsdl_dc/nsdl_dc_v1.02.xsd", "payload_placement": "inline", "payload_schema": ["nsdl_dc"], "doc_version": "0.23.0", "active": true, "resource_locator": "http://www.khanacademy.org/video/subtracting--fractions?topic=developmental-math-1", "identity": {"owner": "Khan Academy", "submitter": "Damon Regan", "submitter_type": "user", "signer": "Damon Regan", "curator": "Damon Regan"}}]}

@jimklo I apologize, but I'm now getting verified: true when I use a different document.  I think this is a user issue.

I think I see the problem.  The 20 minute guide suggests using harvest to download the published document and then verify the document's signature using verify.  The problem is that harvest does not download just the document envelope.  Harvest places an additional harvest envelope around the document envelope, which the verify function seems unable to process.
Obtain provides just the document envelope and when obtain is used to download the document, verify works.
Harvest was used in place of Obtain in the 20 minute guide because of the delay in indexing that was previously associated with Obtain.  @wegrata can you confirm that obtaining by ID (e.g., http://sandbox.learningregistry.org/obtain?request_id=77d1e7a401da4dd7b00806d37ac7d81a&by_doc_ID=T) no longer depends on indexing a CouchDB view and thus should be used in place of harvest in the 20 minute guide?

hmm, @jimklo I upgraded LRSignature to version 0.1.10 per guidance that 0.1.10 includes support for harvested json.  Indeed, harvested json now verifies, but surprisingly obtained json no longer verifies!
I validated that the obtained json is valid json data.  To reproduce:

curl -o harvested.json "http://sandbox.learningregistry.org/harvest/getrecord?request_ID=77d1e7a401da4dd7b00806d37ac7d81a&by_doc_ID=true"

curl -o obtained.json "http://sandbox.learningregistry.org/obtain?request_id=77d1e7a401da4dd7b00806d37ac7d81a&by_doc_ID=T"

type obtained.json | python -m LRSignature.cmd verify --gnupghome "C:/Users/x/AppData/Roaming/gnupg/"
type harvested.json | python -m LRSignature.cmd verify --gnupghome "C:/Users/x/AppData/Roaming/gnupg/"

output:
C:\Development\verify_analysis>type obtained.json | python -m LRSignature.cmd verify --gnupghome "C:/Users/548580/AppData/Roaming/gnupg/"
{"results": [{"doc_ID": "77d1e7a401da4dd7b00806d37ac7d81a", "verified": null}]}

C:\Development\verify_analysis>type harvested.json | python -m LRSignature.cmd verify --gnupghome "C:/Users/548580/AppData/Roaming/gnupg/"
{"results": [{"doc_ID": "77d1e7a401da4dd7b00806d37ac7d81a", "resource_locator":
"http://www.khanacademy.org/video/subtracting--fractions?topic=developmental-math-1", "verified": true}]}


crash upon opening settings

Reported by RageShake: https://github.com/matrix-org/riot-android-rageshakes/issues/2501

Fixed with https://github.com/vector-im/riot-android/pull/2396


Make Linux memory allocations int64 not uint64

The kernel ABI to these values is a string, which accepts the value -1
to mean "unlimited" or an integer up to 2^63 for an amount of memory in
bytes.
While the internal representation in the kernel is unsigned, this is not
exposed in any ABI directly. Because of the user-kernel memory split, values
over 2^63 are not really useful; indeed that much memory is not supported,
as physical memory is limited to 52 bits in the forthcoming switch to five
level page tables. So it is much more natural to support the value -1 for
unlimited, especially as the actual number needed to represent the maximum
has varied in different kernel versions, and across 32 and 64 bit architectures,
so determining the value to use is not possible, so it is necessary to write
the string -1 to the cgroup files.
See also discussion in

https://github.com/opencontainers/runc/pull/1494
https://github.com/opencontainers/runc/pull/1492
https://github.com/opencontainers/runc/pull/1375
https://github.com/opencontainers/runc/issues/1421

Signed-off-by: Justin Cormack<EMAIL_ADDRESS>

LGTM

LGTM

@wking adjusted the wording and line breaks as suggested, and fixed the example which had 0 limits rather than unlimited or unspecified.

On Fri, Jun 23, 2017 at 03:20:57PM -0700, Justin Cormack wrote:

… and fixed the example which had 0 limits rather than unlimited
or unspecified.

You'll also want to make that fix in [1,2].

@wking amended

LGTM

LGTM

See my comments in https://github.com/opencontainers/runc/pull/1492#issuecomment-310807978 , the only benefit we can gain from "uint64 -> int64" is that it makes runc code easier, all other reasons are not making int64 better than uint64, just that int64 is acceptable. I don't know if that's kind of compromise we should make in OCI.
But since there's no real world issue for using int64 for now, let's not back and forth on this anymore, not before 1.0, just want us on the same page on this.
LGTM


Alpha blend two images

Two new PixelImageOperations were introduced that blends two images together according to a mask.
The mask describes the blending of the foreground onto the background. Either two PixelImage[RGB] are passed with a separate mask as PixelImage[Double] or the foreground contains an alpha channel that is used as mask.

Looks convenient, thank you. Can you add a one-line comment to indicate what alphaBlending does and what mask is?


Todo testing

Here:

We used  jest framework for testing.
We create a file by the name of functions.test.js
We used the mock storage.

Happy coding 🎈🎉😊

@julie-ify
Hello,
Thank you for reviewing my project!
According to requirement I have made functions.test.js to test add and remove functions that they really work.
Can you tell the exact mistake I have did?

You only need to add tests for Add and Remove for Part 1 Testing,
Actually you add edit and clear all functionality too!
Remove for part 1!
On Thu, 22 Jun 2023, 5:24 pm Najibullah Jafari, @.***>
wrote:

@julie-ify https://github.com/julie-ify
Hello,
Thank you for reviewing my project!
According to requirement I have made functions.test.js to test add and
remove functions that they really work.
Can you tell the exact mistake I have did?
—
Reply to this email directly, view it on GitHub
https://github.com/najibullahjafari/To-do-list/pull/7#issuecomment-1602546099,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4LZRBTJPHOUWMVT5CBBYRLXMQ2P5ANCNFSM6AAAAAAZQATALI
.
You are receiving this because you were mentioned.Message ID:
@.***>


Fix HTML typo (“stylsheet” → “stylesheet”) for case studies

This makes the case studies cascading stylesheet take effect. It doesn't seem to have a negative impact when I try the fix.

/kind cleanup

/lgtm
/approve
nice catch.


Contacting CA takes too much time, and it times out.

Recently I am experiencing an issue while contacting ACME for certificate renew.
[root@prod1 ~]# time curl https://acme-v01.api.letsencrypt.org/directory
{
"B9GMAHh9hjQ": "https://community.letsencrypt.org/t/adding-random-entries-to-the-directory/33417",
"key-change": "https://acme-v01.api.letsencrypt.org/acme/key-change",
"meta": {
"terms-of-service": "https://letsencrypt.org/documents/LE-SA-v1.2-November-15-2017.pdf"
},
"new-authz": "https://acme-v01.api.letsencrypt.org/acme/new-authz",
"new-cert": "https://acme-v01.api.letsencrypt.org/acme/new-cert",
"new-reg": "https://acme-v01.api.letsencrypt.org/acme/new-reg",
"revoke-cert": "https://acme-v01.api.letsencrypt.org/acme/revoke-cert"
}
real	2m6.353s
user	0m0.086s
sys	0m0.082s
It takes more than 2 minutes, so Caddy timeouts doing that. I want to propose a config option that can configure timeout value on contacting CA. Thanks.
1. What version of Caddy are you using (caddy -version)?
Caddy 0.10.11 (non-commercial use only)
2. What are you trying to do?
Renew my TLS certificate from LetsEncrypt
3. What is your entire Caddyfile?
(paste Caddyfile here)
oreon.feroko.com:443 {
proxy / localhost:3002
gzip
}

4.  Please paste any relevant HTTP request(s) here.

6. What did you expect to see?
Continue serving the site without an error.
7. What did you see instead (give full error messages and/or log)?
[root@prod1 ~]# ./caddy
Activating privacy features...2018/03/18 11:44:35 get directory at 'https://acme-v01.api.letsencrypt.org/directory': failed to get "https://acme-v01.api.letsencrypt.org/directory": Get https://acme-v01.api.letsencrypt.org/directory: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)
8. How can someone who is starting from scratch reproduce the bug as minimally as possible?
Maybe use some network hacking

Use the -catimeout CLI flag.


Can i use script-message to show and give the display seconds?

Hello, i often use toggle mode so i set
open_toggles = true
playlist_display_timeout = 86400
Now i want to call it from other script and I use below command to show the playlist
mp.commandv("script-message", "playlistmanager", "show", "playlist")
But because i set a long timeout the playlist will not disappear automatic
Is it possible to set the display seconds without change the setting?
ex.I can use script-message to call it for 3sec display, and i still can use the original hotkey to call it in toggle mode.
Thanks for your watching.

Done 9f9e067
mp.commandv("script-message", "playlistmanager", "show", "playlist", 2)

Let me know if it works for you. Right now it only changes the timeout for the first render of playlist. That means that if you interact with the playlist or it renders again from some event then it will change to the default timeout. This can be changed if it doesn't suit your usecase.

Thanks for your quickly reply. I try but it seems not work.
If i set playlist_display_timeout = 5 and use
mp.commandv("script-message", "playlistmanager", "show", "playlist", 1)
Playlist will show for 5 secs not 1sec.
I use it on the betterchapters.lua
https://github.com/Argon-/mpv-config/blob/master/scripts/betterchapters.lua
And i want to change the mp.commandv("script-message", "osc-playlist") to your playlistmanager
So i need to show playlist temporary for a timeout.
Thanks for your watching.

Add more test results.
As i say i use in betterchapters.lua that it will show playlist when change to next or previous file.
And i found that the code works if it's only one file, will show the playlist for 1sec.
mp.commandv("script-message", "playlistmanager", "show", "playlist", 1)
But if more then one file, when change to next or previous file will show the playlist for the timeout in setting.
PS.I try to use in keybind that works.
Ctrl+u script-message playlistmanager show playlist 1

Sorry for more test, it seems not work when it occur on file change.
ex.
There ar two files in playlist.
With betterchapters.lua, when i press the previous chapter hotkey at the first chapter of first file
it will show the playlist for the secs i give in script-message.
But when i press the next chapter hotkey at the last chapter of first file
it will go to the second file and show the playlist for the timeout in config file
The same results, when i press the previous chapter hotkey at the first chapter of second file
it will go to the first file and show the playlist for the timeout in config file
Thanks for your watching.

Fix 84cec7e9ad680276c401cdeb28c2c3193de9e5b4
The timeout should now be 1 second only until playlist disappears. Let me know if there's any other issues.

I test and it's work perfect, thanks for your quickly reply


Black screen returned with a vengeance

Getting black screen while audio still playing even if I sign out. I know this is an older issue yet our previous solution won't work

Mine is stuck on a spinning loading icon with a black screen no audio or playback. It works singed out in a incognito window though.

fixed in PR : https://github.com/TheRealJoelmatic/RemoveAdblockThing/pull/4756, Lets hope the maintainers push the code changes


Questions about local development

Hi, I am a newbie for golang, and trying to contribute new features for flink deployers.
What trouble me mosts is that all the dependencies for flink deployers has been included inside  the project itself,  why we still import the dependency using github.com/ing-bank/flink-deployer/cmd/cli/flink instead of flink-deployer/cmd/cli/flink ?
When I want to add new file to the dependencies, I have to change a lot of code about import to make it run as I expected.
Looking forward to your reply.

Hi
First of all, thank you for wanting to contribute.
Checking in the vendor/ folder is actually common in Golang projects. We use dep to manage our dependencies.
There's an article describing the problem you're facing (among others) and it also offers a solution which includes changing the Gopkg.toml file in the project.
I hope this solves your problem. Feel free to add the solution to the README.md if you plan on contributing.

Thanks!  the article is exactly what I need.


Git, command line, and editor don't know where they are

Issue Type: Bug
Description first: I was in VSCode, and attempting to git push, but could not. I was getting errors about various things, ultimately meaning that I was not pushing from the correct location in my hierarchy. So I went to the command line to attempt to push, and that also failed. I also switched directories using the command line to the correct directory, and still was getting errors about not being able to push. So I  switched my entire window to the correct directory, then I could do the push from the CL with no problems.
I have a Visual Studio solution in VSCode, which VSCode does not treat the same way as Visual Studio does, which leads to confusion about which part of the directory structure actually will respond correctly to git commands via the command line.
So in my case, from the editor window I'm in a directory above the .sln file, and two directories above the project directory. From this location, I can't do a git push (via the UI) because the project directory is what needs to be pushed. So I switched to the terminal, and cd solution.slnDirectory/ProjectDirectory and then run git, and it still fails.
Then I change the editor so that the top level directory is the ProjectDirectory, and then I was able to push via the command line.
So basically, when I run git from the command line, git thinks I'm at the top level window in the UI, even though I'm not at the level in the command line.
VS Code version: Code 1.60.2 (7f6ab5485bbc008386c4386d08766667e155244e, 2021-09-22T11:59:27.195Z)
OS version: Darwin x64 20.6.0
Restricted Mode: No

System Info

Item
Value

CPUs
Apple M1 (8 x 24)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
16.00GB (0.04GB free)

Process Argv
--crash-reporter-id c334bbd5-69b6-4ee0-9bdf-260924daed9c

Screen Reader
no

VM
0%

Extensions (40)

Extension
Author (truncated)
Version

vscode-icalendar
af4
1.0.1

ng-template
Ang
12.2.0

swagger-viewer
Arj
3.1.2

npm-intellisense
chr
1.4.0

jshint
dba
0.11.0

vscode-eslint
dba
2.1.25

githistory
don
0.6.18

gitlens
eam
11.6.0

EditorConfig
Edi
0.16.4

tslint
eg2
1.0.47

vscode-npm-script
eg2
0.3.22

prettier-vscode
esb
9.0.0

nasc-touchbar
fel
1.12.0

git-project-manager
fel
1.7.1

cshtml-tm
fir
1.0.0

remotehub
Git
0.14.0

gc-excelviewer
Gra
3.0.44

beautify
Hoo
1.5.0

rest-client
hum
0.24.5

csharpextensions
jch
1.3.1

angular-essentials
joh
12.0.0

Angular2
joh
12.0.0

vscode-peacock
joh
3.9.1

Angular-BeastCode
Mik
10.0.3

prettify-json
moh
0.0.3

csharp
ms-
1.23.15

mssql
ms-
1.10.1

remote-ssh
ms-
0.65.8

remote-ssh-edit
ms-
0.65.8

vscode-typescript-tslint-plugin
ms-
1.3.3

debugger-for-chrome
msj
4.13.0

debugger-for-edge
msj
1.0.15

angular2-inline
nat
0.0.17

angular-console
nrw
17.11.2

material-icon-theme
PKi
4.10.0

vscode-commons
red
0.0.6

vscode-yaml
red
0.22.0

yo
sam
0.9.3

aspnet-helper
sch
0.6.4

autoimport
ste
1.5.4

(1 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
pythonvspyt602:30300191
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492:30256859
pythonvspyt639:30300192
pythontb:30283811
pythonvspyt551:30345470
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30358480
pythondataviewer:30285071
pythonvsuse255:30340121
vscod805:30301674
pythonvspyt200:30340761
binariesv615:30325510
vsccppwt:30364497
pythonvssor306:30344512
bridge0708:30335490
pygetstartedt2:30371810
dockerwalkthrucf:30370837
bridge0723:30353136
pythonrunftest32:30373476
pythonf5test824:30373475
javagetstartedc:30364665
pythonvspyt187:30373474
pydsgsc2:30361791

Hey @lszomoru - thanks for having a look at this. It looks like I started with the built in git commands in VS Code, and then switched to the integrated terminal. I guess my expectation is that VS Code will not make git available in the UI if you're in the "wrong" folder - one that does not have git enabled. But the VSC git in the UI didn't "know" that the folder it was in didn't have git enabled at that point in the hierarchy, so it just failed.
Example:
Editor window - shows MySource as the top level directory. Git commands are available in the UI.
Below MySource is another folder - MySolution - which holds MySolution.sln
Below MySolution is the MyProject Folder which holds the .csproj file. This is where git is enabled. This is the directory I had to use the CLI to cd to for git to work, from the CLI, as it didn't work from the UI built-in commands.
I hope that isn't too confusing.

@rhires, would you be able to create a screen recording (https://gifcap.dev) as that would clarify which folder you have opened in VS Code, and which Git repository was detected by VS Code. Thank you very much in advance!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!