RLP-665 Apply PyLint to the lumino node

Removing unused imports

@JONAF2103 i will wait until the test cases report to review this PR.


Update instructions for Jest versions 15+

There was a question on Gitter about using Enzyme with Jest, and I noticed that the Jest guide was a little out of date. This PR updates that guide for Jest 15 and up.
The guide referenced a sample test repo by Leland Richardson. Rather than ask him to manage updating the samples, I forked it and updated my fork. This change references my updated fork.
I also left the his instructions for earlier versions of Jest that require mocking, just in case someone still has to use an earlier version.

Looks like the build was broken before my PR; I'm not sure how to fix it on my end.

@vjwilson sorry about that; rebase on latest master and the tests should pass.

@vjwilson i've rebased for you; the build should pass now.

Thanks, @ljharb, I was out for a while.


How to decompress the compressed DirectX file format

I am trying to load a compressed DirectX file format.
The structure of this compressed X file is as follows.
First, the first 16 bytes are the header.
The next 4 bytes represent the size of the decompressed X file including the header.
After that one or more data blocks are arranged.
The structure of the data block is as follows.
The first two bytes represent the size after decompression of that compressed block.
The next two bytes represent the size of that compressed block.
After that, it is compressed block in MSZIP format.
Currently I succeeded in decompressing the first data block, but I have failed to decompress after the second one.
I attach a sample file and a sample program that performs decompression.
DecompressedXFileTest.zip
First, "test1.x" is a file which succeeds in decompression.
The result after the extension "test1.result.x" matches the expected result "test1.expectedResult.x".
Next, "test2.x" is a file that fails to decompress.
"test2.result.x" which is the result after elongation does not match the expected result "test2.expectedResult.x".
The difference between "test1.x" and "test2.x" is whether the number of data blocks is one or more.
How can I properly decompress "test2.x"?
(This issue is related to #418 .)

This problem was resolved with the following pull request.
https://github.com/leezer3/OpenBVE/pull/291


Provision to add google adsense

Thanks for a lovely theme gcushen. I have a request, it would be great if there can be some sort of provision to add google adsense code to hugo website with the help of config.toml. Thanks.

Glad you like the theme :) Advertising doesn't really fit the philosophy of the theme. However, you can add advert code by inserting the HTML into your content files or by using Hugo's inheritance principle - copy one of Academic's layouts and paste it in a similar path from your root e.g. your_website_root/layouts/partials/footer_container.html

Thanks for the response. I will try doing it.

@gcushen I am using your theme and i love it. Thank you
It would be good if we could also target the empty left/right are of the posts when viewed on the destktop, without requiring significant changes to the html/css files.
How would you approach this?


Support to print star history and visitor count

Codecov Report

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

@@            Coverage Diff            @@
##             master      #14   +/-   ##
=========================================
  Coverage          ?   28.49%
=========================================
  Files             ?        1
  Lines             ?      179
  Branches          ?        0
=========================================
  Hits              ?       51
  Misses            ?      120
  Partials          ?        8

Flag
Coverage Δ

unittests
28.49% <0.00%> (?)

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

Continue to review full report at Codecov.

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


scoping resource associations with state

Hi Lee,
Thanks for the great set of libraries, it has greatly simplified writing a jsonapi compatible application.
Frequently I find myself scoping queries through some sort of user state, for example in this simplified situation:
class Foo < ApplicationRecord
  has_and_belongs_to_many :bars
end

class Bar < ApplicationRecord
  has_and_belongs_to_many :foos
  has_many :users
end

I might want to scope all requests to the Bar index endpoint, to only return bars that are associated to the current_user. However, if I request GET /foo?include=bar I will still get other Bars associated to the Foo that are not relevant to the user.
How would one accomplish this sort of relationship filtering with the jsonapi_suite? My initial thought would be to allow passing some sort of global scope into render_jsonapi which can be used by the association scope in the FooResource, similar to how ActiveRecord handles scopes:
class FooResource < ApplicationResource
  type :foo

  has_many :bars,
    foreign_key: { foos_bars: :foo_id },
    scope: lambda |current_user| { Bar.where(user: current_user) },
    resource: BarResource
end

class FooController < ApplicationController
  def index
    render_jsonapi Foo.all, current_user: @current_user  # could be generalized to any state
  end
end

What are your thoughts on how this could be handled? I'm happy to implement a PR.

Hey @bastianl, thanks for the kind words. I think there's a better answer here, but wanted to respond while I have a quick minute.
One, you could pass this in with the request, since 0.6.x supports association filtering:
/foos?include=bar&filter[bar][user_id]=123

The filter is applied as normal, just like if we were writing the resource for /bars?filter[user_id]=123:
allow_filter :user_id

I think that's your best immediate-term solution.
For a longer-term one, we actually have access to the controller from the resource. I use this for guards, currently:
allow_filter :whatever, if: :allowed_for_user?

def allowed_for_user?
  # code with current_user here
end

That code is here: https://github.com/jsonapi-suite/jsonapi_compliable/blob/master/lib/jsonapi_compliable/scoping/filterable.rb#L14. resource.context[:object] is the controller.
So one way to give access to the current user is to evaluate all the scope procs within that context. That said, I'm not thrilled about the pattern and not sure I want to double down on it that much.
I think there's a better solution, but that's what's top of my head at the moment.

I have used the filter aspect of the jsonapi successfully in the past, but it seems lacking especially when dealing with has_and_belongs_to many tables, and don't want to expose the intermediate join table to your client. In above example filter would work, but suppose Bar HABTM users! Then there wouldn't be an id to query off without exposing an intermediate table. My actual use case is slightly more complicated :/
class Foo < ApplicationRecord
  has_and_belongs_to_many :bars
end

class Bar < ApplicationRecord
  has_and_belongs_to_many :foos
  has_and_belongs_to_many :bazs
end

class Baz < ApplicationRecord
  has_and_belongs_to_many :bazs
  has_and_belongs_to_many :users
end

I want to to query Foo, but only ever include the Bars that I filter through the Baz - user association, for the current user.. Would love to discuss this with you further next week if you have time. Enjoy your weekend!

HABTM is definitely a use case that can be improved. I've thought a lot about this and the short answer is, "it's on the list, but not a priority for me".
I agree ideally the join table is not exposed, since this is an implementation detail. However, let's say you now add a column to the join table - a boolean like primary or an integer like rank. Since this join table was omitted in the original design, we now have to expose it to access this data and break backwards compatibility.
Really we want this data treated as metadata of the relationship, not a separate object. JSONAPI does support per-object metadata...but almost no library actually supports it and there's not a fully-fleshed out filter/sort/etc paradigm for it. I think it's a good long term direction but not one I'm ready to spend time embracing at the moment.
There is a solution for your use case, though, I think:

In above example filter would work, but suppose Bar HABTM users!
All that said, I'm personally exposing my join tables at the moment. Yes, it's a little more verbose but at least everything is dead simple and easy to reason about. I'm working to release an isomorphic js client that works similar to ActiveRecord...I'm considering if adding a through to the client, just like ActiveRecord, is the real solution here. Would be simpler (if less ideal/correct) than a server-side solution.
All that summed up - definitely on my radar, but I don't think exposing the join tables is prohibitive at the moment, just less than ideal. So I'm waiting to get more data before settling on a solution there.

Another solution you might want to consider is middleware setting User.current via Thread.current. This has fallen out of vogue recently but I've never had much of a problem with it and would be another solution to your original use case. I don't think you should have to do it that way, but something to consider.

Thanks for the help! The filter block should actually do nicely, that wasn't on my radar. I'll check it out and close the issue soon.


Upload logos to Asset Manager when a logo is added in Whitehall

This was added in https://github.com/alphagov/whitehall/pull/3464.


i updated my name to the names.txt file

i simply updated my name to the name.txt file

Please I am sorry for all the trouble we have caused you but count it all joy. Your PR is about to be merged and you will be able to see your name on this url.
To enable me to do this, READ the README again carefully, update your code with the latest and push the update. I am here for you for the next 6 hours before I close the PR so you can initiate a fresh PR which will be easier to merge.


fix: add playlist property to playlist tracks

Changes
Fixes an issue that caused playlist songs to not be added to the queue.
This issue happens when the user calls the play function with the first song from the playlist as a parameter instead of the playlist itself.
Also fixes issues that might arise from not being able to access the playlist property on the tracks when the audioTracksAdd event is emitted.
Status

[x] These changes have been tested and formatted properly.
[ ] This PR includes only documentation changes, no code change.
[ ] This PR introduces some Breaking changes.

LGTM! I honestly have no idea how I missed this.


Add Ability to Set the Browser's URL for a Content Post

Which scope/s are relevant/related to the feature request?
content
Information
The path for a blog post in src/content/blog is the filename that comes after /blog. So, src/content/blog/2023-06-27-test.md generates a path of www.mysite.com/blog/2023-06-27-test.
However, I have an attribute in the frontmatter of my blog posts called custom_slug. I need that slug to be used for the path in the browser's URL. I was able to get that working, basically, but the file could not be found because the custom slug doesn't match the file path to any file.
Describe any alternatives/workarounds you're currently using
I created a branch to demo what I was thinking could work, but I'm not sure if it's correct. Here's a link to it:
https://github.com/pjlamb12/analog/commit/80e04b47949fa3942bd8326395f81587885483e6
The other option is to allow you to pass a parameter for the slug (in my case I passed custom_slug and it found the correct attribute) and then also a filename parameter so that the default slug can be passed and the file can be found.
I would be willing to submit a PR to fix this issue

[X] Yes
[ ] No

I'm running into some errors serving the blog-app, though, so I can't test this. I ran nx serve blog-app, but get the following output:
https://app.warp.dev/block/euQyOFSbLGUxHiO6NlEJww

My demo change listed above most definitely does not work. I'm trying to figure out how to do this, but the problem is that the contentFiles that are injected into the injectContent function is a record where the key is the filename.
I'm gonna keep looking into options.

I think that in the provideContent method that's put in the providers array we would want to declare what the custom slug that will be used is, so that the contentFiles record can be created properly.

I figured it out! But I've run into a tiny issue I haven't figured out yet.
In the terminal, I see all the data being found as expected. This is the console.log of the post$.pipe():

But in the browser, it says this:

I'm not sure why it's working in the server part but not the browser. So I need to figure that out. But once that piece can be figured out, I think this feature will work.

Here's the branch, and you can look at the most recent commit to see the changes I've made, and the issue I'm having.
https://github.com/pjlamb12/analog/tree/content-custom-slug-demo

I also just thought about how in the injectContent method we can use the injection token instead of making the user pass in thecustomSlugAttribute. When I worked on that file at first I didn't have the injection token.

It turns out that once I cleared the .vite folder in node_modules, the browser showed the content again. I will have a PR for this soon.


Move package under scoped npm packages

Hey @hzoo since most of the Babel deps were move to @babel/ scope I don't see why we shouldn't move this package as well.

This is done!


'Price|Agriculture|***|Index' indexed to incorrect year in 'AIM' scenarios

The units on Price|Agriculture|Non-Energy Crops and Livestock|Index and Price|Agriculture|Non-Energy Crops|Index say "Index (2010 = 1)", but the AIM data appear to be indexed to 2005.

@sfujimori, can you please double-check?

Thanks @kvcalvin for reporting this issue and thanks @shinichirofujimoriKU for confirming.
This will be fixed as part of release 1.1.


Add new meteorological forcing data type, sunshine duration (h) and C++ APIs.

@Sibada -
Thank you for issuing a pull request. However, I don't think we will be able to accept this feature. The MTCLIM meteorologic forcing pre-processor has been depreciated in VIC 5 (#288) which we will be releasing shortly. In the future, we hope you will continue to make contributions to VIC. Next time please inquire with us ahead of time to check on their applicability.


fix(Notifications): fix next link for /notification/expired

This PR has already been merged to develop. Just creating a record of the changes so we can look at them later.


Add experimental getExisting method to DurableObjectNamespace

Right now this isn't implemented in workerd's ActorChannel, as this is an internal-only experimental method.
Still need to add a commit to hide getExisting from being generated by workers-types
@MellowYarker as primary reviewer

I noticed there are a couple legit build failures in the PR that got masked by the CI failures we were seeing due to github's changes to tarball hashes.  I'm going ahead and pushing a correction for them.


conjugate heat transfer

Hi,
this solver is really great. I was wondering if you would have an example of conjugate heat transfer for a cylinder in the cross flow... and also show how one can calculate the Nusselt's number around the cylinder..
Thank you

Thanks! We don't yet support conjugate heat transfer, but it is absolutely on the to-do list. It would be a matter of combining this package with ViscousFlow.jl


Add cubic spline interpolation header and example

Hi,
This is my first time attempting to contribute to an open source project, so my apologies for my lack of experience.
What I attempting to do is to add the cubic spline interpolation modules in this project.
Basically it's the same module as PythonRobotic cubicspline planner.
But i think the algorithm itself is not path planning, but an interpolation polynomial, therefore I've added a new workspace called Polynomial.
On the side note, seems the algorithm relies on linspace function, so I've created one in the common.h. The unit test for the linspace is included. Also, one thing iIve notice is somehow the deg2rad function causing compile error due to multiple definition, so i added inline to it.
Please let me know anything i can help to make the code better..

Hi, thank you so much for contributing. It's weird that deg2rad was causing problems, it really shouldn't need to be inlined. I'll have to look into that.
I am approving the CI pipeline and will review the files as soon as I have more time!

Codecov Report

Merging #14 (f8bcf33) into master (f5044a3) will increase coverage by 22.50%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           master      #14       +/-   ##
===========================================
+ Coverage   47.05%   69.56%   +22.50%
===========================================
  Files           2        2
  Lines          17       23        +6
===========================================
+ Hits            8       16        +8
+ Misses          9        7        -2

Impacted Files
Coverage Δ

include/robotics/common.h
100.00% <100.00%> (+100.00%)
: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 f5044a3...f8bcf33. Read the comment docs.

I forgot to mention that in my case, the multiple definition build error occurs when I'm attempting to build everything at once.  It doesn't happen in standalone build.
Please let me know if I should remove the inline from the pull request. Also if I need to modify any other things from the code (other than the @codecov-commenter checks)

The code doesn't seem to compile at the moment. Regarding the multiple definitions, can you describe the sequence of commands you run when you get the error? I can't seem to reproduce it. Thank you!

The code doesn't seem to compile at the moment. Regarding the multiple definitions, can you describe the sequence of commands you run when you get the error? I can't seem to reproduce it. Thank you!

Sorry for late response. The multiple definition error only appear after I've added the linspace function.
First I ran the following command
❯ cmake -DCMAKE_EXPORT_COMPILE_COMMANDS=ON -S test -B build/test

Then,
❯ cmake --build build/test
[ 20%] Building CXX object CMakeFiles/RoboticsTests.dir/source/common/linspace.cpp.o
[ 40%] Building CXX object CMakeFiles/RoboticsTests.dir/source/linear-control/linear-quadratic-regulator.cpp.o
[ 60%] Building CXX object CMakeFiles/RoboticsTests.dir/source/main.cpp.o
[ 80%] Building CXX object CMakeFiles/RoboticsTests.dir/source/robotics.cpp.o
[100%] Linking CXX executable RoboticsTests
/usr/bin/ld: CMakeFiles/RoboticsTests.dir/source/linear-control/linear-quadratic-regulator.cpp.o: in function `Robotics::deg2rad(double)':
linear-quadratic-regulator.cpp:(.text+0x0): multiple definition of `Robotics::deg2rad(double)'; CMakeFiles/RoboticsTests.dir/source/common/linspace.cpp.o:linspace.cpp:(.text+0x0): first defined here
collect2: error: ld returned 1 exit status
make[2]: *** [CMakeFiles/RoboticsTests.dir/build.make:145: RoboticsTests] Error 1
make[1]: *** [CMakeFiles/Makefile2:155: CMakeFiles/RoboticsTests.dir/all] Error 2
make: *** [Makefile:146: all] Error 2


Decoupling and refactoring

Our task order is currently static. Some of our tasks need to be performed on a state change. My thoughts that turn into ramblings:

We should always try to collect level up rewards on exp change
We should always check for badge responses (on each heartbeat?)
Our API requests should be detached from bot decisions (store what we want locally, but only send recent when a response is required)
Move emit calls to centralized place (store emit requests from each task). Allows for easier configuration, decouples task config from log config.
Move delay times and sleep features to centralized place. Same as above, we repeat too many (similar kinds of) waits and make our config entirely too long.
Too many parts of code that are trying to accomplish the same thing. Between catch_pokemon, pokemon_catch_worker, map_move_to_pokemon we repeat much of the code. Plus map_move_to_pokemon changes position and initiates capture attempts (and checks balls). All capture attempts should be force through our catch_pokemon task first (with refactor to implement order logic of map_move_to_pokemon). pokemon_catch_work should be the only one checking balls, our walker should be the entry point for all changes to position (with etc logic changes).

I'll stop for now. My idea is, some files do too much that other files should and some files just too much in general.

Re api calls.. In an ideal universe the bot would only call the api when absolutely necessary, ie. To perform an action for the most part. As far as  things like player stats and inventory,  the bot should grab that info at start, store it in a database, then only refer to the database for future info. Sanity check every so often just to make sure nothing is wrong.
Just my ramble

We should always check for badge responses (on each heartbeat?)
check awarded badges is currently in the heartbeat.

My PR #4746 stored the awarded badge in the bot, and also updated the player_data on every heartbeat. so we could remove the get player_data api calls from some tasks if last 10 second (give or take delays) is good enough.

@Gobberwart @duttonw I love it.

I make some debug over api calls. There is a lot of calls like get_player/get_inventory one after another without delay or delay <= 1s.
Looks like:
bot_task()
-> function get_inventory at 0x107a1bc80
-> function get_inventory at 0x107a1bc80
-> function get_player at 0x107a1bc08
-> function get_inventory at 0x107a1bc80
bot_task()
-> function get_player at 0x107a1bc08
-> function get_inventory at 0x107a1bc80
-> function get_inventory at 0x107a1bc80
-> function get_inventory at 0x107a1bc80

Do we really need this calls every time on bot_task switch?

Ok, some logs here:
[2016-08-29 13:51:17] [MoveToFort]
2016-08-29 13:51:18 (<function get_inventory at 0x10c9e5c80>)
2016-08-29 13:51:18 (<function get_inventory at 0x10c9e5c80>)
2016-08-29 13:51:18 (<function get_inventory at 0x10c9e5c80>)
2016-08-29 13:51:18 (<function get_inventory at 0x10c9e5c80>)
[2016-08-29 13:51:18] [UpdateLiveStats]

get_player should really be removed from tasks and pull from the bot as the
heartbeat keeps it up to date every 10 seconds. (other than the tutorial
task).
tasks calling get_inventory should be using the centralized inventory which
should only refresh every so often. This should be ready to go forward
since recycle now updates internal inventory.
On 29 August 2016 at 20:53, Nikolay Spiridonov<EMAIL_ADDRESS>wrote:

Ok, some logs here:
[2016-08-29 13:51:17] [MoveToFort]
2016-08-29 13:51:18 (<function get_inventory at 0x10c9e5c80>)
2016-08-29 13:51:18 (<function get_inventory at 0x10c9e5c80>)
2016-08-29 13:51:18 (<function get_inventory at 0x10c9e5c80>)
2016-08-29 13:51:18 (<function get_inventory at 0x10c9e5c80>)
[2016-08-29 13:51:18] [UpdateLiveStats]
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/PokemonGoF/PokemonGo-Bot/issues/4754#issuecomment-243093276,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB7XcARCNXD-T2G49I7JHoFYpdhjhpHtks5qkro-gaJpZM4Jtx9x
.

Seems we have to refactor all api.get_inventory() calls to refresh_inventory() with decorated cache object. But what TTL we should use for api call?

I've just submitted a PR which goes some way towards refactoring to reduce api calls. Metrics and UpdateLiveStats now share the same cached inventory information (note that "inventory" from an API perspective includes player_stats so they're now all kept together in inventory)

Removed another api call from incubate_eggs. As far as I can see, GET_INVENTORY is now only called as part of inventory_refresh, which is called by heartbeat. Everything else should just use whatever is cached.

@mjmadsen close?


Add name & description to plugin POM

Reviewer checklist

[ ] Ensure relevant issues are linked (description should include text like "Fixes #")
[ ] Ensure correct labels applied
[ ] Ensure any appropriate documentation has been added or amended

Coverage remained the same at 98.16% when pulling 657550f9cedbbd4bbd8d6a30aab512cffa1fd208 on fix_pom into 5336da0f96cc456186d033f49adda29ab0b9cde7 on main.


Auto balance and sticky drive

Still needs PID controller tuning

This pull request is very old and contains features we no longer want, I will open a new one and only write the features we still need


REST: add round information to box response

Summary
Provide for the round information relevant to the box contents in the indexer box endpoint
Test Plan
Altered saved endpoint response test resources for boxes to include the round number.

I wasn't sure specifically how to generate the fixture response json from the readme so I edited the test_responses by hand

v2/account gives you the round number in the response and validator uses this to see if results from algod and indexer are aligned on the same round.


Integration Test - Release - 2022/04/24 07:27:41

Auto create for integration test. IssueCommandFlow

#pri1

Just type '#pri3'

#pri2

#pri3

#assign PRMergerTest1

#assign NonAllowedUser

The following users don't have valid GitHub IDs, or are not collaborators on this repo: nonalloweduser

#reassign PRMergerTest2

#reassign NonAllowedUser

GitHub user PRMergerTest2 has been removed from the MicrosoftDocs organization, so they were automatically removed as an assignee.

The following users don't have valid GitHub IDs, or are not collaborators on this repo: nonalloweduser

#please-close

#unassign PRMergerTest1

#unassign NonAllowedUser


test: add unit tests for recently-updated commands; minor fixes

updated to most recent cli lib (includes minor change to calling forAllOrganizations function
added unit tests forchannels:metainfo, channels and drivers commands
minor refactoring for unit testing

Types of Changes

[ ] 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 change)
[x] I have read the CONTRIBUTING document
[x] My code follows the code style of this project (npm run lint produces no warnings/errors)
[ ] Any required documentation has been added
[x] I have added tests to cover my changes

:tada: This PR is included in version 1.8.2 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Modal open/close race condition

Current Behavior
I am using a Modal to fetch some user input. The modal calls a custom response callback when the "submit" button is clicked.
Then, in the response callback, I send the data to the API, and await the result. When the API returns an error, I open a new Modal.
However, the new modal gets "closed"/hidden immediately. When I open any new modal, it shows up again, before the one that was "newly requested".
The API introduces a delay - If there is no delay, it works as expected (similar to the "queue multiple" example in the docs).
Expected Behavior
The new modal should open and not get closed.
Steps To Reproduce
No response
Link to Reproduction / Stackblitz
No response
More Information
No response

Here is some example code:
const modal: ModalSettings = {
	type: 'component',
	title: 'Custom Component',
	component: 'CustomComponent',
	response: async (r: any) => {
		if (r) {
			await new Promise(r => setTimeout(r, 20));
			showErrorModal(modalStore, "Test");
		}
	}
};
modalStore.trigger(modal);

Hello,
I was looking at this issue.
Something like this works
function nestedModal(): void {
		const c: ModalSettings = {
			type: 'prompt',
			title: 'Enter Name',
			body: 'Provide your first name in the field below.',
			value: 'Skeleton',
			response: async (r: string) => {
				if (r) {
					await new Promise((r) => setTimeout(r, 20));
					modalStore.close();
					setTimeout(() => {
						showNextModal();
					}, 1000);
				}
			}
		};
		modalStore.trigger(c);
	}

Just to chime in here - I would probably recommend a promise or timeout approach to avoid the race condition for now. But the we're taking a very different approach to modals in Skeleton v3. So if this is something you need or want changed for Skeleton v2, I might suggest submitting a PR if you can. Otherwise the suggestion above should hopefully allow you ti bide your time until transitioning to v3.

Do you want to updated the v2 documentation for this specific racing condition or in general nested modals?

Just to chime in here - I would probably recommend a promise or timeout approach like Null is showing to avoid the race condition for now. But the we're taking a very different approach to modals in Skeleton v3. So if this is something you need or want changed for Skeleton v2, I might suggest submitting a PR if you can. Otherwise the suggestion above should hopefully allow you ti bide your time until transitioning to v3.

Do you want to updated the v2 documentation for this specific racing condition or in general nested modals?

No, I wouldn't imagine this is a use case that comes up often enough it needs to be documented. But we can obviously keep this thread for reference if the odd support issue does come around. My hope is that once v3 launches, most folks will move to that - especially given that Svelte 4 is a dead end road.


Not possible to see the code from a runtime packages even "includeSourceInSymbolFile": true

1 .Describe the bug
When debugging OnPrem Customers is not possible to see code from runtime packages even "includeSourceInSymbolFile": true was used in function Get-BcContainerAppRuntimePackage.
2. To Reproduce
We are using the next configuration for resourceExposurePolicy
"resourceExposurePolicy": {
"allowDebugging": true,
"allowDownloadingSource": true,
"includeSourceInSymbolFile": true
}
I used this command to get the runtime package
Get-BcContainerAppRuntimePackage -containerName $ContainerName -appName $AppName -appFile "$Destination$PackageName" -IncludeSourceInPackageFile:$true
3. Expected behavior
Code should be included (exposed) in the symbols of a downloaded runtime package when the package was created with -IncludeSourceInPackageFile $true parameter
4. Actual behavior
After the runtime package is published, when downloading symbols the objects are missing the code, not possible to add brake points, see the code from functions in debug mode , etc.
5. Versions:
•	AL Language: v13.1.1065068
•	Business Central application:  "<IP_ADDRESS>",
•	List of Visual Studio Code extensions that you have installed:
AL Language

From PowerShell is working as expected if we fallow the next steps :

Export the .app with Get-BcContainerAppRuntimePackage
Unpublish/uninstall the app
Install the runtime package resulted from Get-BcContainerAppRuntimePackage using , Publish-NAVApp, Sync-NAVApp and Install-NAVApp
Create a PTE
Download symbols
We can see the code - OK

We are using this command also part of the pipeline that is generating the environment, and in this case when we create a PTE and Download symbols  we are not able to see the code.
Is any difference in using the function manually (by using a PTE described above) or from a pipeline?

There is no difference whether you run code locally or in a pipeline.
I am not entirely sure what the difference is though - in both samples, you are using Get-BcContainerAppRuntimePackage to get the runtime package - are you saying that the runtime package you are getting is different???
In example #1 - are you trying to use the runtime package as symbols in the symbols folder in VS Code?

In the end I think is a problem in our pipeline, because when I do the steps manually is working fine.


Add FileDownloadDelegate for simple file downloads

A simple test is included, let me know if you'd like it to be more sophisticated.

As a quick note, I'd like to get more opinions on whether the feature itself should be included before I address the technical feedback provided by Cory.

Hi @artemredkin, do you think that this feature makes sense overall in AsyncHTTPClient? Just wondering if it's worth updating the PR to address the feedback, otherwise I'd close it.

@MaxDesiatov Hi! Sorry I missed your question. I do think this feature would be useful

Just as a heads up, the main development branch has been changed to main, following the Swift policy on this.
This PR has been re-targeted to main and should just work. However when performing rebases etc please keep this in mind -- you may want to fetch the main branch and rebase onto the main branch from now on since master is not up-to-date anymore and is going to be deleted shortly.


[GameTest Bug]

Describe the bug
Game test framework hard crashes to desktop when invalid arguments are sent to block.permutation.getProperty()
To Reproduce
in a javascript file subcribe to block place event and add the following lines of code
`
world.events.blockPlace.subscribe((eventData) => {
let tempPerm=eventData.block.permutation.getAllProperties()
console.warn(tempPerm)//works
for(var i =0; i<tempPerm.length;i++){
    console.warn(tempPerm[i])//gives me an object
    console.warn(eventData.block.permutation.getProperty(tempPerm[i]))//hard crashes the game and doesnt print anything to the console also doesnt write console log of previous tasks (ie the file write never completes and the content log in the logs folder is empty.
}

}
`
Expected behavior
Game test framwork prints an error to the log stating that the argument type object is not a valid input parameter to method getProperty
actual behavior
Minecraft closes
Content log does not write to disk

This bug was fixed some months ago, thank you for you to reporting this issue, you can close it.

Hello!
It looks like we've made some progress on this. Since it's been a while, I'm going to close this issue, but if you have any additional feedback, please reach out to us!
Thanks!


[SPARK-37866][TESTS] Set file.encoding to UTF-8 for SBT tests

What changes were proposed in this pull request?
This PR aims to set -Dfile.encoding=UTF-8 for SBT tests.
Why are the changes needed?
To make the tests robust on various OSs with different locales.
BEFORE
$ LANG=C.UTF-8 build/sbt "sql/testOnly *.JDBCV2Suite -- -z non-ascii"
...
[info] - column name with non-ascii *** FAILED *** (2 seconds, 668 milliseconds)
[info]   "== Parsed Logical Plan ==
[info]   'Project [unresolvedalias('COUNT('?), None)]
[info]   +- 'UnresolvedRelation [h2, test, person], [], false
[info]
[info]   == Analyzed Logical Plan ==
[info]   count(?): bigint
[info]   Aggregate [count(?#x) AS count(?)#xL]
[info]   +- SubqueryAlias h2.test.person
[info]      +- RelationV2[?#x] test.person
[info]
[info]   == Optimized Logical Plan ==
[info]   Project [COUNT(U&"\540d")#xL AS count(?#x)#xL AS count(?)#xL]
[info]   +- RelationV2[COUNT(U&"\540d")#xL] test.person
[info]
[info]   == Physical Plan ==
[info]   *(1) Project [COUNT(U&"\540d")#xL AS count(?)#xL]
[info]   +- *(1) Scan org.apache.spark.sql.execution.datasources.v2.jdbc.JDBCScan$$anon$1@df881a4 [COUNT(U&"\540d")#xL] PushedAggregates: [COUNT(`?`)], PushedFilters: [], PushedGroupByColumns: [], ReadSchema: struct<COUNT(U&"\540d"):bigint>
[info]
[info]   " did not contain "PushedAggregates: [COUNT(`名`)]" (ExplainSuite.scala:66)

AFTER
[info] JDBCV2Suite:
...
[info] - column name with non-ascii (2 seconds, 950 milliseconds)
...
[info] All tests passed.
[success] Total time: 21 s, completed Jan 11, 2022, 2:18:29 AM

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass the CIs.

Thank you, @srowen . All tests passed.

Merged to master.


[Enhancement]:  URL scheme support for NoSQL

Module
MongoDB
Proposal
Hi @rnorth @bsideup
I have been using the JDBC URL scheme to launch test containers
I'm currently working in a project which involves NoSQL mongoDB ,So i would like to know is there support for NoSQL similar to JDBC/RJDBC .If so please share it
Regards,
Faizal

@smfaizalkhan - sorry, I can't answer your question, I am just a simple user and not familiar with this part of the project.

I'm also not involved with the MongoDB module, but if there are features to be implemented across the NoSQL modules I'm happy to help.

@smfaizalkhan can you elaborate more, please? What's your use case and how testcontainers can help? why the current methods such as connectionString are not enough?

Hi @eddumelendez
Just like how how we have for RDBC like
jdbc:tc:mysql:5.7.34:///databasename
i wish we have something on sme lines for NoSQL

There is currently no SPI/driver concept for NoSQL technologies in the Java language, which would make any implementation extremely challenging (and not sure if feasible). Furthermore, to my knowledge, there is also no connection URL standard across NoSQL technologies, similar to JDBC/R2DBC.
And while I can understand that the magic JDBC-Proxy-Driver support in Testcontainers is very useful for some use cases, using Testcontainers via its programmatic API is the most powerful and flexible integration after all and can be used for all use cases.


build-skia fails

When trying to setup the library on macOS for development according to the instructions at https://github.com/Shopify/react-native-skia#building, compile errors occur when building Skia for iOS through yarn build-skia:
build-skia-error.txt
build-skia-error-2.txt
I then ran git submodule update --remote --merge to update the submodules:
diff --git a/externals/depot_tools b/externals/depot_tools
index 5ba5119..db41eed 160000
--- a/externals/depot_tools
+++ b/externals/depot_tools
@@ -1 +1 @@
-Subproject commit 5ba5119227002ab191b0f916df435b7410c4ca3c
+Subproject commit db41eed6b7442273b54bac9695567d97b60718de

diff --git a/externals/skia b/externals/skia
index 0b898c4..b4d28b2 160000
--- a/externals/skia
+++ b/externals/skia
@@ -1 +1 @@
-Subproject commit 0b898c4b8e2a5291d378a99701a533eaf1d4d301
+Subproject commit b4d28b2f35396ae4dd69338254415066629dfd25

After this, Skia compiles and builds correctly, but I don't know if updating the submodules is required.
Maybe there is some step missing in the build instructions?

No, i tried this:

checkout repo
git submodule update --init --recursive
yarn etc.
yarn build-skia

When this failed, I tried git submodule update --remote --merge and then the build succeeded.

Since the Skia build is mainly for library development and the automatic build is working, I'll close this issue for now since you managed to build with your workaround.


Design doc: No init startup for store gateway.

Hi :wave:
Sharing the design doc with some initial discussion around removing or limiting startup time for store gateway to be used potentially by Cortex:

No startup metadata for blocks synchronization.
Loading blocks on-demand on query time.

While this has many benefits it is a tradeoff e.g in query latency for "cold blocks": https://docs.google.com/document/d/1En0Hr1OqZLlsF-_JtpYSWEu2mBXyVYx7BvXoivW0n3U/edit#
For Thanos, I am personally super interested in the latter step: loading blocks on-demand on query time. For block meta files synchronization we can go quite far with compaction and current iterating over the bucket. Those two can be tackled separately as well.
Feedback is welcome!

better to add label like design or plan or something...

There's still some interest from my side on this improvement.

Agree - we should explore that further. We have stale bot exactly for that reason (: To revisit the issue if we are still interested at least once a month.

Let's keep it alive for a bit more.
Food for thought: the current lack of store gateway HA with sharding (if 1 gateway goes down, all queries fail) may actually be solved with a lazy storage which would allow to a fast re-sharding across gateways without downtime (if we completely remove the initial sync delay).

unstale (:
On Sun, 5 Apr 2020 at 12:12, stale[bot]<EMAIL_ADDRESS>wrote:

This issue/PR has been automatically marked as stale because it has not
had recent activity. Please comment on status otherwise the issue will be
closed in a week. Thank you for your contributions.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/thanos-io/thanos/issues/1813#issuecomment-609399565,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABVA3O43ULO5W6L2YSEJJE3RLBRSBANCNFSM4JSXGSWA
.

I initially wrote that design doc but, from my side, the interest decreased since then. The reason is that loading cold blocks on-demand would have a significant performance impact on queries hitting blocks not-yet-loaded and the great work done by @bwplotka to reduce the in-memory index header size as well as the blocks sharding introducing in the Cortex store-gateway relaxed this need.
I still believe we need a faster way to "scan the bucket" to discover new/deleted blocks which doesn't involve having every single query/gateway instance running a periodic full bucket scan, but it's a different topic. Also, the recent support for metadata caching on memcached (including bucket List operation) relaxed this need too.


unable to open playerglobal.swc sometime

and i use the fp23 not the 11.1
Building mymain
mxmlc-cli -load-config="D:\sdk\airsdk25/frameworks/flex-config.xml" -load-config+=obj\mymainConfig.xml -advanced-telemetry=true -swf-version=25 -o obj\mymain636295960251190233
Running java as: java
Loading configuration: D:\sdk\airsdk25\frameworks\flex-config.xml
Loading configuration: D:\proj\my\client\client\Game_Main\obj\mymainConfig.xml
D:\sdk\airsdk25\frameworks\flex-config.xml:47:Error: unable to open 'D:\sdk\airsdk25\frameworks\libs\player\11.1\playerglobal.swc'.
D:\sdk\airsdk25\frameworks\flex-config.xml (line: 47)

Build halted with errors (mxmlc-cli).

From the output you paste the error comes from the compiler itself, not FD. One option could be that FD is run from some user that cannot read that file for some reason, so the compiler is running as that user too. There are other options, like some program blocking the file, you may be able to use Process Explorer and/or Process Monitor to find out the culprit.

http://www.flashdevelop.org/community/viewtopic.php?f=13&t=11053
the same this.maybe can not get libs/player/11.9/playerglobal.swc


new top menu

This is my suggested top menu.
In some point we will need to remove the mouse over effect on the menu links (the logo, issue, and the dropdown trigger buttons). I decided to avoid doing it because that's probably better done by customizing semantic theme. So my tweaks to css are minimal.

User Dropdown Menu

New (repo, organization, ...) Dropdown Menu

Popups

The logo acts as the link / and there is nothing else to do. If is a signed user it will link to dashboard and if it is anonymous user will link to home.

Thanks!


Fixed #15250 -- Avoided extra query on some multi-table inheritance queries.

https://code.djangoproject.com/ticket/15250

Buildbot, add to whitelist.

Thanks for your contribution, this is really elegantly implemented.

You might add "Thanks marekw2143 for the initial patch." in the commit message since that appears to be based on that?

@timgraham Thanks for the feedback.
I believe I've now addressed all your comments.

@charettes Thanks. Pushed the adjustments.

merged in 38575b007a722d6af510ea46d46393a4cda9ca29, thanks!

Very nice patch, great work @czpython


Why is the token table empty?

Hello!
Registered through the VK two accounts. Why is the token table still empty?
In social_account, the code column is empty. How to get a token? Is this possible or should it be received separately?
Foto: ibb.co/VmWfCkr, ibb.co/4N7m7rV
Regards!

close

Сlose #1049

close #1049


registry/health: adding healthcheck package

Summary
Package health provides a generic health checking framework. The health package works expvar style. By importing the package the debug server is getting a /debug/health endpoint that returns the current status of the application. If there are no errors, /debug/health will return a HTTP 200 status, together with an empty JSON reply {}. If there are any checks with errors, the JSON reply will include all the failed checks, and the response will be have a HTTP 500 status.
A Check can either be run synchronously, or asynchronously. We recommend that most checks are registered as an asynchronous check, so a call to the /debug/health endpoint always returns
immediately. This pattern is particularly useful for checks that verify upstream connectivity or database status, since they might take a long time to return/timeout.
Installing
To install health, just import it in your application:
import "github.com/docker/distribution/health"
You can also (optionally) import health/api that will add two convenience endpoints: /debug/health/down and /debug/health/up. These endpoints add "manual" checks that allow the service to quickly be brought in/out of rotation.
import _ "github.com/docker/distribution/registry/health/api"
# curl localhost:5001/debug/health
{}
# curl -X POST localhost:5001/debug/health/down
# curl localhost:5001/debug/health
{"manual_http_status":"Manual Check"}

After importing these packages to your main application, you can start registering checks.
Registering Checks
The recommended way of registering checks is using a periodic Check. PeriodicChecks run on a certain schedule and asynchronously update the status of the check. This allows CheckStatus() to return without blocking on an expensive check.
A trivial example of a check that runs every 5 seconds and shuts down our server if the current minute is even, could be added as follows:
 func currentMinuteEvenCheck() error {
   m := time.Now().Minute()
   if m%2 == 0 {
     return errors.New("Current minute is even!")
   }
   return nil
 }

 health.RegisterPeriodicFunc("minute_even", currentMinuteEvenCheck, time.Second*5)

Alternatively, you can also make use of RegisterPeriodicThresholdFunc to implement the exact same check, but add a threshold of failures after which the check will be unhealthy. This is particularly useful for flaky Checks, ensuring some stability of the service when handling them.
health.RegisterPeriodicThresholdFunc("minute_even", currentMinuteEvenCheck, time.Second*5, 4)

The lowest-level way to interact with the health package is calling Register directly. Register allows you to pass in an arbitrary string and something that implements Checker and runs your check. If your method returns an error with nil, it is considered a healthy check, otherwise it will make the health check endpoint /debug/health start returning a 500 and list the specific check that failed.
Assuming you wish to register a method called currentMinuteEvenCheck() error you could do that by doing:
health.Register("even_minute", health.CheckFunc(currentMinuteEvenCheck))

CheckFunc is a convenience type that implements Checker.
Another way of registering a check could be by using an anonymous function and the convenience method RegisterFunc. An example that makes the status endpoint always return an error:
 health.RegisterFunc("my_check", func() error {
  return Errors.new("This is an error!")
}))

Examples
You could also use the health checker mechanism to ensure your application only comes up if certain conditions are met, or to allow the developer to take the service out of rotation immediately. An example that checks database connectivity and immediately takes the server out of rotation on err:
 updater = health.NewStatusUpdater()
  health.RegisterFunc("database_check", func() error {
   return updater.Check()
 }))

 conn, err := Connect(...) // database call here
 if err != nil {
   updater.Update(errors.New("Error connecting to the database: " + err.Error()))
 }
```

You can also use the predefined Checkers that come included with the health package. First, import the checks:

`import "github.com/docker/distribution/health/checks`

After that you can make use of any of the provided checks. An example of using a `FileChecker` to take the application out of rotation if a certain file exists can be done as follows:

```go
health.Register("fileChecker", health.PeriodicChecker(checks.FileChecker("/tmp/disable"), time.Second*5))
```
After registering the check, it is trivial to take an application out of rotation from the console:
```bash
# curl localhost:5001/debug/health
{}
# touch /tmp/disable
# curl localhost:5001/debug/health
{"fileChecker":"file exists"}
```
You could also test the connectivity to a downstream service by using a `HTTPChecker`, but ensure that you only mark the test unhealthy if there are a minimum of two failures in a row:

```go
health.Register("httpChecker", health.PeriodicThresholdChecker(checks.HTTPChecker("https://www.google.pt"), time.Second*5, 2))
```

cc/ @NathanMcCauley @endophage @stevvooe

Let's move this package into distribution/health. I want to avoid having registry-specific functionality in this package.

@stevvooe @NathanMcCauley would love another review.

cc @icecrime @aluzzardi @mavenugo because this kind of stuff has a broader interest.

Got a few comments, but overall I think it's cool :+1:

I'm not sure if the addition of github.com/stretchr/testify is justified. Equality in Go has levels of subtlety that should be communicated (most languages do, but we ignore it). This subtlety should be a part of the test.
Let's examine implementation implementation of assert.Equal to prove my point:

Use direct equality if one is nil:

  if expected == nil || actual == nil {
  	return expected == actual
  }

Not terrible, and it makes sense.

Then, we go to reflect.DeepEqual:

  if reflect.DeepEqual(expected, actual) {
  	return true
  }

This will walk the instance tree, resolving pointers and checking slice elements.

If you're just looking to reduce line count, we can define this function:
func checkErr(t *testing, err error, format string, args...interface{}) {
  if err != nil {
    t.Fatalf("unexpected error " + format, args...)
  }
}

If we want to use this everywhere, we can define CheckErr as part of testutil. A similar function could be defined for checking a response.

We are looking good here. The only items I really care about:

Remove use of testify package.
Use of 500 for error status. We should differentiate between unexpected errors and health-based down conditions. 500 error code should only ever be unexpected (we don't follow this everywhere, but we should).

LGTM!
Nice work on this one.

LGTM. Might be nice to be able to configure the checkers but that can be future work.
...
health:
  checkers:
    filechecker: [/path/one/, /path/two/]
    httpchecker: [http://foo.com, http://bar.com]
...

@endophage white-belt changes?

Sounds good to me

ping @vieux @abronan
Is this something we could use?

@aluzzardi Please! Let us know if you need help!


Harmony.js

All configs of harmony.js with custom ip list are timeout or unavailable

Harmony only takes information from you and configures with them, it does not have anything of its own. If your configurations are not working, it is because of your own mistake. Now I do not know if you entered the wrong UUID or you made a mistake in one of the 6 sections where you were supposed to replace the hostname, please check again.


Breaking DDL initialization ordering

When ddl migration reapplying via QuartzDataSourceScriptDatabaseInitializer it's not executes completely.
The reason is DDL structure and DDL ordering: when Spring executes quartz's DDL it can't delete qrtz_triggers table, because there is present foreign key qrtz_retry_cron_triggers => qrtz_triggers.
It cause incorrect scheduler's behaviour.

Fixed in https://github.com/Fin-Ex/quartz-retry/commit/5f25e5d435c35cdb53e21a8bf65418beb2888434


How to send props from navigator to it's screen without disrupting the navigator.navigation in that particular screen

When I do something in my stack navigator
export const NewsFetch = StackNavigator({ Brief     :  { screen :  props=> <NewsColumnFixed head='bangladesh' /> }, Detail   :  { screen : NewsDetails },                });
the default navigator props is not passing down or somehow my props named head overwriting the default props and so I can not navigate from one screen to another using navigator.navigate()
Thanks in advance.

Find the solution after a long hustle I found this on a Cooment of @joshuapainter
in this issue https://github.com/react-community/react-navigation/issues/876
His Code
import React from 'react';
import {
AppRegistry,
Text,
} from 'react-native';
import { StackNavigator } from 'react-navigation';
class MyApp extends React.Component {
render() {
console.log('this.props in MyApp', this.props); // This will list the initialProps.
// StackNavigator **only** accepts a screenProps prop so we're passing
// initialProps through that.
return <Navigator screenProps={this.props} />;

}
}
class HomeScreen extends React.Component {
render() {
console.log('this.props in HomeScreen', this.props);
// This will output something like this:
// { screenProps: { ...your initialProps }, navigation: { ...StackNavigator stuff... } }
return <Text>My initialProps are {JSON.stringify(this.props.screenProps)}.</Text>;

}
}
const Navigator = StackNavigator({
Home: { screen: HomeScreen },
});
AppRegistry.registerComponent('MyApp', () => MyApp);

can you guys please give a solution which works for react-navigation-3 in which you must set up your app container directly.


Jonmv/deployment playground 2

@freva plepase review. @ldalves FYI :)

The playground code is WIP, and can probably be improved a bit in various areas :)


fix: retain pod spec volume when its name has default token prefix

In clusters prior to version 1.24, the default volume name mounted for service accounts is random.
And pod spec volumes can not been updated, so we retain it.

Can you found out the reference commit?

Can you found out the reference commit?

It appears that there is no fixed name for it, and modifying the pod.spec.volumes is not allowed. Therefore, we will retain the pod.spec.volumes.


Issue with ProfileCopyright Field

When I run the following code there is no python issue however, the exif that is generated has a weird occurrence. I believe that ProfileCopyright in the exif is a UTF-8 byte array. When I encode the field, I just get back the integer values of the text I have inputed. I am sure that I am just misunderstanding how to encode this information. Any help is most appreciated. Sample code below.
import piexif
from PIL import Image

def manipulate_exif(img, remove_gps):

    author = 'Brian Levin'
    license = '{} 2016, All Rights Reserved'

    exif = piexif.load(img.info['exif'])
    exif['0th'][piexif.ImageIFD.Artist] = author
    exif['0th'][piexif.ImageIFD.XPAuthor] = bytearray(author, 'utf-16')

    # Line with issue
    exif['0th'][piexif.ImageIFD.ProfileCopyright] = bytearray(license, 'utf-8')
    # End Line with issue

    exif['0th'][piexif.ImageIFD.Copyright] = license

    return piexif.dump(exif)

img = Image.open('./IMG_2223.jpg')
exif_copy = manipulate_exif(img, False)
img.save('./Test.jpg', 'JPEG', exif=exif_copy)

This produces the following exif statement for ProfileCopyright:
ProfileCopyright: 123 125 32 50 48 49 54 44 32 65 108 108 32 82 105 103 104 116 115 32 82 101 115 101 114 118 101 100

It can't use UTF-8 string for exif value, so encode and decode between UTF-8 string and byte array  by yourself.

I believe my current code does this and the result is not human readable. Can you give me a code example that would put a human readable value in profilecopyright?

Hmm... like below?
"".join([chr(s) for s in profilecopyright])

Just the tip.

I mean in the actual exif. I can use exiftool to set a human readable value in ProfileCopyright. Can I use piexif to set this to a human readable value in the exif? Or are you saying that it will always be a block of integers if I use piexif?

Type of ProfileCopyright is "Byte". It's as tuple of byte in piexif.

Right, I get that. When I write XPAuthor as a utf-16 per the standard, I use to following code and it works as expected. It converts the string author into an array of bytes using the utf-16 character set.
exif['0th'][piexif.ImageIFD.XPAuthor] = bytearray(author, 'utf-16')
However, when I want to write ProfileCopyright as a utf-8 per the standard, I use the following code and it produces abnormal results in the exif. It converts the byte array of utf-8 into a string literal of integers.
exif['0th'][piexif.ImageIFD.ProfileCopyright] = bytearray(license.format(author), 'utf-8')
The piexif library is not behaving as expected. It is producing an incorrect result.

Can you paste here input and result both UTF8 and UTF16?

I am running Python 3.5.2 but I have also tested the above script in Python 2.7.12.
I have wrapped the bytearray in a tuple per your instruction and it made no difference. This is the bytearray output from the script that is fed into the exif.
utf8: (66, 114, 105, 97, 110, 32, 76, 101, 118, 105, 110, 32, 50, 48, 49, 54, 44, 32, 65, 108, 108, 32, 82, 105, 103, 104, 116, 115, 32, 82, 101, 115, 101, 114, 118, 101, 100)

utf16: (255, 254, 66, 0, 114, 0, 105, 0, 97, 0, 110, 0, 32, 0, 76, 0, 101, 0, 118, 0, 105, 0, 110, 0)

The utf16 works as expected. The utf8 does not work.
Here is the script to generate the about output.
import piexif
from PIL import Image

def manipulate_exif(img, remove_gps):

    author = 'Brian Levin'
    license = '{} 2016, All Rights Reserved'
    utf8_bytearray = tuple(bytearray(license.format(author), 'utf-8'))
    utf16_bytearray = tuple(bytearray(author, 'utf-16'))

    print('utf8: {}'.format(utf8_bytearray))
    print('utf16: {}'.format(utf16_bytearray))

    exif = piexif.load(img.info['exif'])
    exif['0th'][piexif.ImageIFD.Artist] = author
    exif['0th'][piexif.ImageIFD.XPAuthor] = utf16_bytearray

    exif['0th'][piexif.ImageIFD.ProfileCopyright] = utf8_bytearray
    exif['0th'][piexif.ImageIFD.Copyright] = license.format(author)

    return piexif.dump(exif)

img = Image.open('./IMG_2223.jpg')
exif_copy = manipulate_exif(img, False)
img.save('./Test.jpg', 'JPEG', exif=exif_copy)

Here is the image that I ran the script on.

Here is the result of the script:

img_2223.exif.txt

Piexif returns ProfileCopyright value as "(int, int...)". Isn't it a exiftool's matter?
import piexif
from PIL import Image

author = 'Brian Levin'
license = '{} 2016, All Rights Reserved'
s = bytearray(license.format(author), 'utf-8')
e = {"0th":{piexif.ImageIFD.ProfileCopyright:s}}
exif_bytes = piexif.dump(e)

i = Image.open("a.jpg")
i.save('b.jpg', exif=exif_bytes)

exif1 = piexif.load("b.jpg")
print(exif1["0th"][piexif.ImageIFD.ProfileCopyright])
"".join([chr(s) for s in exif1["0th"][piexif.ImageIFD.ProfileCopyright]])

i2 = Image.open("b.jpg")
exif2 =  i2._getexif()
print(exif2[piexif.ImageIFD.ProfileCopyright])

No, I get the same result (a list of integers) with iPhoto, Photos, Aperture, Lightroom, Photoshop, http://regex.info/exif.cgi, and every other program that I have that reads exif data. However, if I use exiftool to set that value they all return a normal (human readable text) result and not a string of integers.
So either they are all wrong in how they are reading and writing the exif or there is something wrong with the way piexif is reading and writing the exif.

I have filed a bug report over at Exiftool to see if we can determine in which code base the issue exists.
http://u88.n24.queensu.ca/exiftool/forum/index.php/topic,7424.0.html

Have you compared list of integers value between correct exif jpg and wrong exif jpg in piexif?

Yes, the text is correct when it is back converted from integers to a string. However, it should be text not integers in the exif and it should not require back converting to text.
The crux of the issue is when I use exiftool to modify that text it is text and not a string of integers. When I use Piexif it is a string of integers. Either Piexif isn't writing the data correctly to the exif or exiftool is not reading it correctly.
You stated the issue was with Exiftool. Posted an issue ticket over there to make sure that is not the case.
My guess is the issue is with Piexif because Exiftool can also write to the exif and it produces the correct result.

Upload correct exif jpg here.

I guess that if type is UTF8, the value should be append some flag.

Here is a correct exif for the ProfileCopyright field.

Here is a dump of the exif.
Test_Correct.jpg.exif.txt

Conclusion is that exiftool and piexif have no bug. There is a specification difference between two lib.
Exiftool supports "ProfileCopyright" as string(http://search.cpan.org/~exiftool/Image-ExifTool-10.20/lib/Image/ExifTool/TagNames.pod).
In piexif, it is as tuple of byte. Piexif references Exiv2 tags(http://www.exiv2.org/tags.html).

You are correct. It is fixed in the next version of exiftool to match your specification.
See http://u88.n24.queensu.ca/exiftool/forum/index.php/topic,7424.0.html

To understand specification is good way to understand matter.


Supress nodes in config debug output

Debug output of variables is currently very long due to the fact that all nodes variables are included under 'nodes'
This PR compresses other nodes:
The output becomes:
INFO[0000] p1 vars = {
      "links": [
            {
                  "far": {
                        "ip": "<IP_ADDRESS>/31",
                        "isis_iid": "0",
                        "name": "to_p1",
                        "node": "asbr1",
                        "port": "1/1/c2/1",
                        "sid_idx": "3"
                  },
                  "ip": "<IP_ADDRESS>/31",
                  "isis_iid": "0",
                  "name": "to_asbr1",
                  "port": "1/1/c1/1",
                  "sid_idx": "8"
            },
            {
                  "far": {
                        "ip": "<IP_ADDRESS>/31",
                        "isis_iid": "0",
                        "name": "to_p1",
                        "node": "abr1",
                        "port": "1/1/c1/1",
                        "sid_idx": "2"
                  },
                  "ip": "<IP_ADDRESS>/31",
                  "isis_iid": "0",
                  "name": "to_abr1",
                  "port": "1/1/c2/1",
                  "sid_idx": "8"
            }
      ],
      "node": "p1",
      "nodes": "{p1: {...}, pe1: {...}, pe2: {...}, abr1: {...}, asbr1: {...}, asbr2: {...}, }",
      "r20": "1",
      "role": "vr-sros",
      "systemip": "<IP_ADDRESS>/32"
  }

The important part here is
      "nodes": "{p1: {...}, pe1: {...}, pe2: {...}, abr1: {...}, asbr1: {...}, asbr2: {...}, }",

which makes the total output about 1/7th of what it is today.
Note that with the debug flags -d -c 1 you do see the other node individually as well
It is also possible to see the full output using a debug count of 3 or higher -d 3

yes, collapsing the nodes of the vars is a good choice
I also have the following comment about the --check flag. I do not think it makes sense to require a user to specify the number of lines of rendered config they want to see. I think by default we should display the full rendered config when --check is supplied.
It would be awesome to allow to dump the rendered configs as files as well as just printing them on the console.
Like I always want to see the full rendered config to make sure that my templates work as expected, and so far I need to artificially crank up the number of displayed lines and doesn't look right

I suggest we keep check as a count, but by default print all lines, so:

-c would be all lines (your use-case & the simplest option)
-c 1 would be 1 line of config

So keeping both of our use-cases. I use it a lot to just show the variables that I will have available, like this
clab config -t topo.clab.yml -c1 -d

I use it a lot to just show the variables that I will have available

that looks like using abusing the flag  =)

I'm open for suggestions. What about

-c /-c1 show rendered template
-cc /-c2 show the vars
-ccc, where c>=3 show all

what if --check | -c becomes a stringvar that can take

-c -- show rendered template (default string value is "template")
-c vars -- show vars
-c all -- show vars and full template

Hi @hellt , I'm having some reservations about the filter I added here and think this should be a dedicated flag
To debug we could rather have an action flag, options here are: vars, template/dry-run, compare, commit. and possibly a show/send (implies no transaction - this is based of the template name today, which is probably not ideal)
Taking this into account, I believe we need to split the filter and the action, then we end up with something like this:
Flags:
  -p, --template-path strings   comma separated list of paths to search for templates
  -l, --template-list strings   comma separated list of template names to render
  -a, --action string           specify the configuration action: (default "commit")
                                - vars: only show variables that will be used by the templates
                                - template: show rendered templates
                                - commit: render the template and commit to the node
                                - compare: render the template and compare with the current configuration
                                - send: send the command without any transaction (use for show commands)
  -f, --filter strings          comma separated list of nodes to include

What do you think?

Maybe better to have subcommands for that? i.e.

clab config send
clab config template
clab config compare

this will be more idiomatic and in line with the way we use Cobra to structure other CLI commands
the flags -p, -l, -f are global to config command, so they will propagate to subcommands
I wouldn't make a distinction between commit and send actions, because in a generic ssh transport (that I plan to add eventually and be based on scrapligo) the commit command will be part of the template to let user decide which commit command they need (commit [confirmed, timeout, now, stay, etc])

Once you move the transaction awareness from clab into the template, I believe you will loose the flexibility to have a template, start playing with compare and when you are happy commit.
You will see that all transactional awareness start with /configure global and discard to ensure you are starting from scratch,- do you want to move all this responsibility to the template user?

yes, there soon will be a scrapli-cfg available in Go that will does the smart diff/compare. I am planning to use that, as this will deliver multivendor support.
But now let me check with @carlmontanari if scrapligo-cfg intends to make a decision which commit command to use...
@carlmontanari in scrapli-cfg, does a user provide the config snippet with commit bla command, or scrapli-cfg appends this?

There is no way for a computer to know your intention, compare/commit

the intention is given with a command clab config compare vs clab config send/commit
at this moment (while scrapligo doesn't have cfg subsystem in place) I am more than happy with your approach (just using subcommands instead of a flag)

My understanding of the -cfg submodule is that you will have to pass a parameter to scapligo-cfg's commit/compare/send (send being the default or rather scapligo as it is today, without cfg)
Will look into the subcommands, so we will end up with 5 of these subcommands? vars, template, compare/commit/send? Won't this mean a bit of code duplication?

I am thinking about having the following commands

config - this does the commit operation
config with --send will send the commands in raw mode without entering into any config mode or committin on behalf of a user?
config template - this will render the templates without sending them anywhere. potentially this command will be able to have local flags to save rendered configs under per-node directory for example.
if config template is provided with local --vars flag, it will show only the vars used for templation
config compare - does the compare

so in total three commands maybe?

What about 2 only:

config (None / compare / send)
config template

In this case the --send flag wont be carried forward unnecessarily to config template and all three variants of 1 has to do with actually transmitting to the node.
2 would require new parameters for saving to files etc


Word ending in "REF" prevents submission of an XML draft

Describe the issue
A user reported their XML draft submission was being rejected with the error, Expected an XML file of type "application/xml" or "text/xml", found one of type "text/plain". Submitting the text version of the draft was successful.
The XML was being rejected because libmagic was reporting its file type as "text/plain". This turned out to be caused by the word "PREF" appearing in the document followed by whitespace. Any word ending in "REF" followed by whitespace triggers this problem.
It seems likely the workaround to fix #4453 is causing this overzealous identifications of "REF " as a marker of the ASCII text file type.
A small example that triggers the error:
<?xml version="1.0" encoding="utf-8"?>
<t>AWORDENDINGINREF </t>

(this should be rejected as an invalid draft, not because it's not an XML file)
Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

OBE - fixed by #6500


Support sending a pipeline of redis commands.

Hello! Here's another patch, it let's you send a bunch
of commands together without waiting for each one.
You get the results back as a list of all the results.
I added a new function eredis:qp for this, but I'm happy
to use a different name or convention, just let me know.

Hey, thanks for the patch! I fixed an issue with concurrency, if there are multiple commands in the queue your patch added the pipelined request to the end of the queue, which was incorrect if you are waiting for more than just a single command in the queue.
I wanted to get everything into a branch on eredis proper, but due to user error it ended up in master. I would very much appreciate some battle-testing if you are able to do so, before it we proclaim it done.

The pipeline changes are now in it's own branch called "pipeline", please try it out.

Good catch, thank you! I tried it out with a large number of processes in parallel, works great.

So what do you think, ready to go into trunk?

I would like to do some more testing first, mainly adding another basho bench driver to do pipelined requests (for every write, do 100 pipelined requests for example), but I haven't had time yet. If you want us to move faster, this would help ;)

Sounds good, I will see about doing that.

Ok, I sent you a separate pull request with a basho bench driver.


Add Zarf build version when inspecting packages

A little quality-of-life update to make it easier to be confident what package you are deploying (especially when iterating on building a new example package).
I wrestled with the idea of putting the Zarf version the package was built with somewhere in the conformation "Deploy this Zarf package?" prompt but decided against it.

/test all

/test all


correct module layout for gradle desktop example

Currently the module-layout for the gradle example and compose-desktop is "gradle-jvm", which is wrong.
Please update the example, that other people can use it without any hassle.

Thank you for spotting this @Beutlin! I believe in that case it would be better to just entirely remove the section about the layout.

Well that was quick! Thanks again. I will backport these changes to our internal main branch so it is there for the next release too.

@joffrey-bion I wonder why tests didn't catch that

@amakeev this is because our tests only run assemble in this project, and basically built nothing successfully. I tried to modify them to actually run the compose app, but that brings up a UI and gives an infinite Gradle build, so it's not suitable to use this in a test case.


Vs code is not launching at ubantu 18.04

Try running from Terminal... If you've installed from tarball you might be missing libraries.
In case you can't figure out what to install, install apt-file, update it (like you would apt or apt-get) and use apt-file find <filename>

@mfocko great advice!
@rjptechsolutions also how did you install VS Code?


✨[feature Request]: Implement Backend with Appwrite

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
Implement a complete backend for the Retro app using Appwrite to manage user authentication, product management, cart functionality, order management, and user reviews.
Use Case
This backend will provide a robust infrastructure for handling user data and interactions, making it easier to manage products, track orders, and enhance user engagement.
Benefits
Users will be able to sign up and log in to the app, add products to their cart, place orders, and leave reviews—all seamlessly integrated through the Appwrite backend.
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24EXTD contributor
[X] I have starred the repository

@Atharva-04  go ahead.


Behat 4

Because the issue of bumping dependencies has been raised it's worth suggesting that 3.2 could be 4.0 (or both could be released together)
4.0 would be a good opportunity to also bump PHP versions and remove depredations.

I don't feel like we exceeded the capability of 3.x branch and I also do not yet have a clear deprecation plan. So maybe after 3.2 or 3.3.


Add Netty Based HTTP2 implementation for storage server.

Add AsyncWritableChannel interface for MessageReadSet and Send.
Add StorageServerNettyFactory and StorageServerNettyChannelInitializer.
Add HTTP2 availability in AmbryServer but it is disabled unitl HTTP2 port populated in Helix.
Add ServerHttp2Test.

@cgtz  Ready for merge.


CDDP: Logging

(Thurston)

UPDATES:

Tested the config_driven_data_pipelines wheel file in Databricks.

Confirms that the logging features works fine as expected.

@gary918, what else shall we check? Shall we could close this task? Thanks for your confirmation. :-)


removeTrack fails with 'Sender was not created by this connection.'

Browsers and versions affected
Chrome: Version 59.0.3071.115 (Official Build) (64-bit)
adapter.js: 4.2.0
Description
Calling removeTrack causes the exception 'Sender was not created by this connection.'.
The code that's failing is:
var isLocal = sender._pc === pc;
if (!isLocal) {
throw new DOMException('Sender was not created by this connection.',
'InvalidAccessError');
}
sender._pc is undefined. The only place _pc appears to be set is in the routine shimGetSendersWithDtmf.
Steps to reproduce
Call removeTrack
Expected results
Track should be removed without throwing an exception
Actual results
An exception is thrown.

how are you getting the sender? Ideally, can you show a fiddle that reproduces the issue?

@DanEdwards61 removeTrack expects a sender, not a track.
Though adapter should give a better error. Here's a fiddle to reproduce it.
In Chrome (w/adapter): InvalidAccessError: Sender was not created by this connection.
In Firefox: TypeError: Argument 1 of RTCPeerConnection.removeTrack does not implement interface RTCRtpSender.
Looks like we need an
if (!(sender instanceof RTCRtpSender)) throw new TypeError(msgLikeInFF);

I mean the same is true for most other polyfills, but given replaceTrack's confusing name, I think a PR is warranted here.

will be fixed in #625 with a better error and test. Thanks!


Azure Pipeline Validation failed with "The signing feature is not available for your organization yet." at step "Generate BOM"

Describe the bug
Pipeline Validation failed with "The signing feature is not available for your organization yet." at step "Generate BOM".
I have retried for several times and get the same failure.
Exception or Stack Trace
You can get the detail failure in this PR: https://github.com/Azure/azure-sdk-for-cpp/pull/4664
And the failed pipeline: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=2815063&view=logs&jobId=7c6ba154-1dd3-507d-6d24-8db90b0372c9&j=7c6ba154-1dd3-507d-6d24-8db90b0372c9&t=01245f84-53cd-51f5-7f46-ca8f9eb2dd03

@RickWinter @danieljurek  I'm not sure if azure-core or  engsys team should be assigned this issue.

Assigning to @danieljurek, but I suspect that this may end up somewhere in 1ES.

From a chat in the EngSys channel, it looks like SBOM is not interesting for PRs. To that end, I think we can exclude SBOM work during PRs.


The handling of jsr310 times (package names start with java.time) is open to customization with dubbo's own spi implementation, which provides a default implementation in spring that allows you to configure date formats

What is the purpose of the change
Specify the jsr310 time format based on the configuration when generic invoke
Verifying this change
org.apache.dubbo.spring.boot.context.event.Jsr310ConverterApplicationListenerTest


ErrorMessage does not show validation error until lose focus

🐛 Bug report
Current Behavior
My validation code is called, sets errors but ErrorMessage does not show the message
Expected behavior
Is this by design ? ( can see that input has to be touched )
Reproducible example
https://codesandbox.io/s/n1r51yk3om
Additional context

Your environment

Software
Version(s)

Formik
1.5.4

React
16.5

TypeScript
yes

Browser
any

npm/Yarn
what codesandbox uses

Operating System
windows

in your code you can see error message only if type "error" in input, cause you validate like this:          if (values.theError === "error") {           errors.theError = "Houston....";         }

@telenord That's right, it was intentional.  If I type 'error' then I validate that there is an error for theError value but the provided  error does not show in the ErrorMessage component until I tab away.  Is this by design ?

@telenord , i think,  cause <ErrorMessage /> used touched prop of field, that fired after blur. You may use
 {props.errors.theError ? <div>{props.errors.theError}</div> : null} instead <ErrorMessage />

@telenord by design then.  Thanks

This seems to also happen if you're using a promise for your validation (messages do not seem to come through until the blur event occurs on the input)(I whole different problem it seems, but same outcomes).
It would be nice if ErrorMessage supported a flag like <ErrorMessage noTouch />


ConvertTo-PnPClientSIdePage reports overwrite previous reports

Category
[ ] Bug
[X] Enhancement
Problem Area
[ ] Page Transformation: Error during the setup/use of the Page Transformation UI solution (did you check our troubleshooting guide?)
[X] Page Transformation: Error during the use of page transformation from PnP PowerShell
[ ] Page Transformation: Error during the use of page transformation from .Net
[ ] Page Transformation: Page is not looking correct after transformation
[ ] Modernization Scanner: something went wrong...
Expected or Desired Behavior
Provide a way to make transformation report file names more distinct. For instance it could include the name of the file being converted. Since files cannot have the same name within a library this should ensure one report does not overwrite another. Maybe also include the seconds in the date as it usually takes a few seconds to process the page.
E.g.:
Transformation-Reports/Page-Transformation-Report-[page-name]-[date]

Observed Behavior
Setting -LogType File or SharePoint produces report files in their respective locations, with file names following a standard pattern: Page-Transformation-Report-[date]. When running multiple migrations as part of a script these report names are not distinct enough causing subsequent reports to override older ones.
Steps to Reproduce
See Observed behaviour
PowerShell PnP Version: 3.16.1912

Including the page name would also make it much easier to connect the correct report to the migrated page.

Hi @rudfoss ,
Thanks for your feedback. The generated reports have a timestamp in them with allows to have a report per second and given that transforming a page will in all cases take more than a second I don't see how you can have reports that overwrite each other. Can you cross check that you actually see reports being overwritten?
About the name: as you can configure the the transformation to "accumulate" the report data of multiple pages in a single report we do not put the page name in the report. Use the -LogSkipFlush parameter on the  ConvertTo-ClientSidePage cmdlet and then when you've transformed your pages call the Save-PnPClientSidePageConversionLog  cmdlet to trigger the logs to persist.

Hi, thanks for the quick reply. I did not know about the accumulation feature. Will definitely check that out. Regarding the name, my reports do not contain the time down to the second, maybe it has something to do with the locale on my machine or site (Norwegian, 1044)?
Regardless, if i can accumulate the reports that is enough for me to resolve the issue. Thanks for the tip! I'll implement it immediately.

Thanks for the reply. I'll close the issue then


Installing Snorkel on Windows (as a package?)

Hello there,
first of all, very nice work, congrats!
I would like to ask if there's a way to 'properly' run Snorkel on Windows.
In the installation instructions, I can see that you are required to run a bash script (run.sh) in order for snorkel to start. Hence, I suppose snorkel cannot really be installed inside python, only launched when working in the correct directory.
I'm trying to run snorkel (more specifically the CDR tutorial) and I've installed cygwin to allow me run the run.sh file on Windows. I can see in the command line:
Snorkel home directory: /cygdrive/c/Users/KrasakisA/Downloads/snorkel-master 0.6.2
which should be fine, however I cannot import snorkelSession or anything, unless I move the ipython notebooks into the root folder (so that os.getcwd changes to the root snorkel folder).
Is there a proper/recommended way to fix this?
Thanks in advance

Hi @littlewine,
Version 0.7, now available on master, is installable completely as a conda package. Would you let us know if it's any easier to get working on Windows? Thanks!

Thanks for the update, but I have been using snorkel on Ubuntu the last couple of months.
I would be happy to try it out on Windows and give you some feedback, but I am currently too busy to do so. However, I'll let you know for sure in case I try it out.

Sounds good!  Closing this for now

@stephenbach There is trouble with installing the package with pip once you create a fresh env of python 3.6.8 or above. Somehow only the labeling module was missing with the rest intact. With conda install, it kept giving an error due to the torch version(I had installed pytorch 1.3 before) and said it Collecting torch<1.2.0,>=1.1.0 (from snorkel)   Could not find a version that satisfies the requirement torch<1.2.0,>=1.1.0 (from snorkel) (from versions: 0.1.2, 0.1.2.post1, 0.1.2.post2) No matching distribution found for torch<1.2.0,>=1.1.0 (from snorkel) , but if you try the readme instructions to install the specific version of torch and then install snorkel it works. Then you simply update your torch version


pam_mount: change order of lines in pam_mount.conf

Change order of pam_mount.conf.xml so that users can override the preset configs.
My use case is to mount a gocryptfs (a fuse program) volume. I can not do that in current order.
Because even if I change the <fusermount> and <fuserumount> by add below to extraVolumes
<fusemount>${pkgs.fuse}/bin/mount.fuse %(VOLUME) %(MNTPT) "%(before=\"-o \" OPTIONS)"</fusemount>
<fuseumount>${pkgs.fuse}/bin/fusermount -u %(MNTPT)</fuseumount>

mount.fuse still does not work because it can not find fusermount. pam_mount will told stat /bin/fusermount failed.
Fine, I can add a <path> section to extraVolumes
<path>${pkgs.fuse}/bin:${pkgs.coreutils}/bin:${pkgs.utillinux}/bin</path>

but then the <path> section is overridden by the hardcoded <path>${pkgs.utillinux}/bin</path> below. So it still does not work.

Motivation for this change
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 nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @flokli

If I understand correctly your problems is you need to change the path but it's hardcoded.
Why not add an option to let users change it?

Yeah. My current problem is that I can not modify path. The curent file also has a few other option specified. I think in rare situation, some other people may want to chang those options.

Looks good. We always want to give manual configuration higher precedence over preset configuration.


Jupyter Lab Desktop Memory issue

Hi,
Could u please tell me how to check memory limitation of the Jupyter Lab Desktop and, how to increase it ?
Thank u

can you provide more details about the issue you are having? any steps to reproduce?

can you provide more details about the issue you are having? any steps to reproduce?

Actually, when i was train my model in jupyter notebook using chrome, I encountered the memory issue due to the limitation of memory . Therefore, I just want to increase the memory size of JupyterLab Desktop

closing since no response for a long time.


[Add] MariaDB

Site URL
https://mariadb.org/
Description of the above provided site
MariaDB Server is one of the most popular open source relational databases. It’s made by the original developers of MySQL and guaranteed to stay open source. It is part of most cloud offerings and the default in most Linux distributions.
Category
Backend
Sub Category
database
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)
[X] I am participating in Hacktoberfest 2023

@haldaranup
I have assigned you the issue. Please proceed with it, and don't hesitate to tag me @Anmol-Baranwal or @CBID2 if you require any assistance.


Is relLangURL non-plain text?

The following snippet only works when plainify is added.
{{ $currentPage := . }}
{{ range .Site.Menus.main }}
{{ .Name  }}
{{ end }}

Is this the intended behaviour?
I did not investigate properly, but probably not only relLangURL may be affected.
Thanks!

This the Hugo bugs issue tracker.
Please ask support questions at https://discourse.gohugo.io
Thanks.

Thank you.
I do not need any support because the solution above is working fine.
Yet, I think the behaviour is unintended and maybe should be corrected (not sure, as stated).

This is a known behavior and not a bug.

Ok. Thanks for your explanation!


[Issue / Improvement] Pushable Blocks cannot be pushed through any debug blocks / light bulbs

Would make sense for at least light bulbs since a lot of maps use them on ground level.

Potential Probelm: Activating debug mode and having a pushable be in a function block is a resulting issue out of making it go through them.
Solution: Pushables need to be set back to their base position in debug mode, and upon deactivating debug mode the block needs to be reverted to where the player pushed it.


[Concurrency] Close a hole that allowed non-Sendable types to cross isolation boundaries in initializers.

Initializing a global actor isolated stored property with a non-Sendable value is effectively passing that non-Sendable value into a global actor isolated context. It is invalid for the initialization to happen from outside the actor, e.g. in a nonisolated init. However, this code is currently accepted
class NonSendable {}

class C {
  @MainActor var ns: NonSendable

  init(ns: NonSendable) {
    self.ns = ns
  }
}

This code passed the actor isolation checker due to the carve outs for diagnostics that are deferred to the flow-sensitive isolation pass, which also doesn't handle non-Sendable types crossing isolation boundaries via stored property initialization. The solution is to only allow those carve outs for Sendable stored property types.

@swift-ci please smoke test

@swift-ci please smoke test


updated bugs from unit tests

Updated a typo, dataone cn connection errors, and set some example documentation to dontrun.

Merging


Vilt image processor

What does this PR do?
Adds vilt image processor
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[x] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Fix render methods to make both components controlled.

External updates to value are now reflected in the dropdowns.
This should address #12

This will not work for the region dropdown if the country is supplied to the region dropdown like this:
<RegionDropdown
  id="formControlsSelect"
  classes="form-control"
  country="US"
  name="region"
  countryValueType="short"
  value={region}
  labelType="short"
  valueType="short"
  onChange={(val) => this.selectRegion(val)}
/>

Because of the code in shouldComponentUpdate, which needs to be something like this, since the country never changes and the component will therefore never be re-rendered when the value changes.
shouldComponentUpdate (nextProps) {
  return (nextProps.country !== this.props.country) || (nextProps.value !== this.props.value);
}

That's the point of controlled components.
The country should come from a source that's always reflecting current state.

Sorry, you're right, I missed your second part.
Did you test your solution? I can add to the PR.

It appears to work..

I did test it, and it worked for me.

Thanks, @bebbi! Your fix looks good. I'm totally swamped tonight but I'll test it out tomorrow and get a fresh version out the door. Appreciated!

Very appreciated @benkeen. And thanks @tchaffee for the collab.


JIT session error. Aborted (core dumped)

I tried this function on 2 computers. One has Ubuntu 22.04 and the other has 20.04. Python version 3.10. Codon version 0.15.5. Cuda is installed on both computers. I don't know why I'm getting this error. Can you help me?
JIT session error: Symbols not found: [ seq_nvptx_memcpy_d2h, seq_nvptx_memcpy_h2d, seq_nvptx_device_free, seq_nvptx_function, seq_nvptx_invoke, seq_nvptx_load_module, seq_nvptx_device_alloc ]
Failure value returned from cantFail wrapped call
Failed to materialize symbols: { (main, {hello:0[std.internal.types.ptr.List[int],std.internal.types.ptr.List[int],std.internal.types.ptr.List[int]], main }) }
UNREACHABLE executed at /github/workspace/llvm/include/llvm/Support/Error.h:786!
Aborted (core dumped)
The function I am running;

@Agymtcelik This issue appears to be similar to #193.

Looks like an issue with CUDA symbols and JIT. Pinging @arshajii


fetchName: Filter outdated names

Let's keep it for now, but we shouldn't rely on this names resolving: https://devchat.aeternity.com/group/aepp-base?msg=Eov97Hv2ZoLjLMrF5


⚠ 366.ru has degraded performance

In 04df32a, 366.ru (https://366.ru) experienced degraded performance:

HTTP code: 200
Response time: 1591 ms

Resolved: 366.ru performance has improved in 3d296df.


Adding ARM storage module and samples

Adding ARM storage module, samples and docs.

/check-enforcer override


Support more than two schemas

The parser throws an error when passing an EDMX file with more than 2 schemas. Is there any specific reason why that's the case? If there are only 2 schemas, they seem to get merged. Shouldn't that be possible for more than 2 schemas as well?
Here's the relevant code:
https://github.com/SAP/cloud-sdk/blob/master/packages/generator/src/edmx-parser/edmx-file-reader.ts#L60

Hi @AndreasHae , thank you for approaching us 😃
your feature request is added to our backlog and will be discussed in the refinement.
I have some additional questions below:

Could you please provide us the .edmx file example with 2+ schemas, if possible?
Where (e.g., SAP API Business Hub) did you get this .edmx file for which system (e.g., S/4, SuccessFactors or some other odata system).
OData version? v2 or v4?
Is it a very urgent requirement?

Thank you.

Hi @jjtang1985, thanks for the quick response.
I cannot provide the actual EDMX file, but its structure resembles this:
<?xml version="1.0" encoding="utf-8"?>
<edmx:Edmx Version="4.0"
  xmlns:edmx="http://docs.oasis-open.org/odata/ns/edmx">
  <edmx:DataServices>
    <Schema Namespace="Company.Product.Namespace1"
      xmlns="http://docs.oasis-open.org/odata/ns/edm">
      <EntityType Name="...">...</EntityType>
    </Schema>
    <Schema Namespace="Company.Product.Namespace2"
      xmlns="http://docs.oasis-open.org/odata/ns/edm">
      <EntityType Name="...">...</EntityType>
    </Schema>
    <Schema Namespace="Company.Product.Namespace3"
      xmlns="http://docs.oasis-open.org/odata/ns/edm">
      <EntityType Name="...">...</EntityType>
    </Schema>
    <Schema Namespace="Company.Product.Namespace4"
      xmlns="http://docs.oasis-open.org/odata/ns/edm">
      <EntityType Name="...">...</EntityType>
    </Schema>
    <Schema Namespace="Company.Product.Enums"
      xmlns="http://docs.oasis-open.org/odata/ns/edm">
      <EnumType Name="...">...</EnumType>
      <EnumType Name="...">...</EnumType>
      <EnumType Name="...">...</EnumType>
    <Schema Namespace="Default"
      xmlns="http://docs.oasis-open.org/odata/ns/edm">
      <EntityContainer Name="Container">
        <EntitySet Name="..." EntityType="Company.Product.Namespace1..." />
        <EntitySet Name="..." EntityType="Company.Product.Namespace2..." />
        <EntitySet Name="..." EntityType="Company.Product.Namespace3..." />
        <EntitySet Name="..." EntityType="Company.Product.Namespace4..." />
      </EntityContainer>
    </Schema>
  </edmx:DataServices>
</edmx:Edmx>

I got the response from a custom-built .NET service using the Microsoft.AspNet.OData library.
The OData version is 4.0.
The requirement is not that urgent, but from what I've seen this should be simple to implement (just remove the if-statement I linked in my first comment).

@AndreasHae, please note that this repository has been moved and the link to this issue has changed.

Hi @AndreasHae , the version 1.33.0 has been released, which supports using multiple schemas. Please try it and let me know whether it solves your issue.

Hi @AndreasHae , I'm going to close this ticket since it has not been updated for 8 days.

Hi @AndreasHae, I'll close this ticket. Please reopen if you still have questions.