Tuning tmpfs in LXD containers

LXD 4.0.1
Ubuntu Bionic
So I should probably have picked up on this long ago, but today it was brought to my attention that LXD containers by default "inherit" the size allocation of tmpfs mounts from the host system. As we have lots of RAM all our unprivileged containers have access to about 189GB of memory directly in /dev/shm, /run and /sys/fs/cgroup
This is no good, as while containers can only exhaust the amount of RAM afforded to them by their profile - they can spill over into swap space and fill up the swap partition with junk, if they wanted to.
Besides 189GB is probably completely overkill - wouldn't 1GB be plenty?
I have been searching around and there are hints out there that this can be tuned in LXD config for my containers but I am unclear how I should achieve this exactly.
I have tried just scaling /dev/shm for starters using an fstab entry on the host, and while it works on the host level it does not take effect inside the container. Besides, this does not seem like the right approach whether inside the container or on the host level.
What is the deal here? Thanks!

tmpfs is indeed constrained by the cgroup limits, so while the user can allocate far more memory than they have, they can't run you out of memory through it.
Looking around, there is no way that I can find to change the default size of a tmpfs for the entire system. It always uses its documented default of half of the size memory.
The container can certainly specify a size= for their various mounts which then makes things look less ridiculous in df and others.
I personally would quite welcome a kernel sysctl to override the default size of tmpfs-es given we're dealing with machines taking TB of RAM these days, but I don't know how this would be received upstream.
@brauner what do you think? Think we should whip up a simple patch to make this configurable through sysctl? I don't see us needing to namespace this at all, for cases like @Kramerican, just having a way to say that all tmpfs mounts default to say 2GB would be good enough.

Closing as not a LXD bug, but if @brauner thinks it's worth a kernel patch and it's not particularly hard to write, we'll put it on our list of kernel work to do to make things better.

Much appreciated @stgraber and @brauner - while they cannot run you out of memory as you say, they could run you out of swap space and create a lot of i/o noise on whatever device holds the swap (usually your OS disk). So it's not a critical issue per-se from my POV, but worrying nonetheless.
In the meantime, could you drop me a hint as to how I would go about modifying size= for container "system mounts" in my config? I gave it a shot trying to figure it out this morning, but Google is not being kind to me today (or the various suggestions and docs I can find are quite unclear on this)
Thank you yet again for the excellent support and tremendous work you guys are doing 👍

I would expect that putting /etc/fstab entries would work, assuming systemd doesn't hardcode those things internally.

Ah I see - well yes that does indeed work (already tried it), but is easily changed by anyone with access to the container, so this would only be cosmetic. I was under the impression you could do config updates on the host level, e.g. adding this to a profile or something like that - but I guess not?

Nope, there's no way to override such things from the host in a way that can't similarly be undone by the user.

Alright I see - I guess we'll just have to hope @brauner thinks this can be patched in for all of us with lots and lots of RAM :D

Decided to try and implement something here today, but turns out modifying /etc/fstab doesn't survive container reboots. If I add something like none /dev/shm tmpfs defaults,nodev,nosuid,noexec,size=1G 0 0 and remount it takes effect, but is ignored on restart.
@brauner Is there anywhere I can "officially" request the aforementioned kernel patch, so this doesn't just vanish at the bottom of a "nice to have" list scribbled on a post-it note somewhere? :O)
I get that this is an absolutely non-critical issue, and I can live with it - but it would be nice to get some control here (mostly due to the theoretical "kill all swap space for everybody" attack contained herein). Thanks.!


what the meaning of the method of forward and backward?

could anyone who can tell me what the use of the method of forward and backward?
thank you in advance~

Hello @zengjie617789! The forward method predicts the action to be taken for any given observation from the given algorithm's policy.
The backward method trains the function approximator (neural network) for a given sample. It also stores the experience replay memory (DQN) for generating samples at the time of learning.

thank you very much ; i feel sorry that i did not read the code carefully;
and i have some doubts that what is the mean of nb_steps_warmsup and the metrics?
does it  mean only the number of steps  exceeds the parameter then the neural network would learn and update the weight?
but if it doesnot, what about the trainning?

You are correct. DQN stores the experience in the memory buffer for the first nb_steps_warmup. This is done to get the required size of batch during experience replay.
Metrics stores the logs of data generated per update. It may be your loss function, reward, output of neural netwok etc. You may want to have a look at https://keras.io/metrics/.

thank you@mirraaj; here is another question , i can not get the idea of the graph in tensorboard ; why is there two neural network？ and there are no connections between them ; and as well as the two loss;
i know the theory of DQN; but i can not understand this graph ,can you help me ? thank you
here is the shoot below:

Please refer to the following.
https://github.com/keras-rl/keras-rl/blob/ee3614a4083094aa540cfd4b43aaf2f4a4db11fa/rl/agents/dqn.py#L196
The first neural network is the network which you define as your function approximator. The second neural network is your trainable network to compute mean square error loss. Please let me know if I fail to explain anything.

@mirraaj  thank you for your patience . I have some questions about the results and i donot know why;
i first trained the agent and get the result graph below:

but for some reasons and it is interrupted, so i trained it again using  the trained weight ,but i get the result below:

i donot know what the reason for it ,do you know why ?
thank you in advance


Use JSON logging format for cert-manager

cert-manager now supports configuring structured logging. Do that, and improve the chart values.yaml documentation. Remove the name overrides, since we don't use them, and simplify the resources we create directly.

This doesn't work yet; we're waiting for cert-manager 1.12.0


test: add unit test for RETURN opcode

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[x] Other (please describe):

What is the current behavior?

No unit test for RETURN at the moment
What is the new behavior?

Add unit test for RETURN opcode (exec_return)

Other information

Thanks @pedrobergamini 🚀


gan_language.py has some reference errors

Hi I wanted to try out your very promising language model but I get the following errors when trying to run gan_language.py
Traceback (most recent call last):
  File "gan_language.py", line 89, in <module>
    fake_inputs = Generator(BATCH_SIZE)
  File "gan_language.py", line 64, in Generator
    output = tf.reshape(output, [-1, SETTINGS['dim_g'], SEQ_LEN])
NameError: global name 'SETTINGS' is not defined

To solve this I simply used DIM instead of SETTINGS['dim_g'] but I'm not sure whether this is the right value. When I then try to run again I get the following error
Traceback (most recent call last):
  File "gan_language.py", line 130, in <module>
    true_char_ngram_lms = [data_tools.NgramLanguageModel(i+1, lines[10*BATCH_SIZE:], tokenize=False) for i in xrange(4)]
NameError: name 'data_tools' is not defined

I believe that data_tools should be replaced with language_helpers

Thanks for catching these! Fixed now.


JAV-276 added chinese documents

Coverage remained the same at 93.75% when pulling 6383bc054bba8d664d97e6975ecad96eb7d311a1 on eric-lee-ltk:JAV-276_added_chinese_docs into 1fe812c9cba9bc56bf68870f4b0e48dab9b42212 on ServiceComb:master.

Coverage remained the same at 93.75% when pulling ddcdfe603516d7adeeddffe411acf52ae864fe54 on eric-lee-ltk:JAV-276_added_chinese_docs into 1fe812c9cba9bc56bf68870f4b0e48dab9b42212 on ServiceComb:master.

Coverage remained the same at 93.75% when pulling 8fbbb3d839be5b155b4dd749aa9c3a1076ea559c on eric-lee-ltk:JAV-276_added_chinese_docs into 1fe812c9cba9bc56bf68870f4b0e48dab9b42212 on ServiceComb:master.


Bind AudioEffectSpectrumAnalyzer::FFT_Size enums

Let godot-cpp compile after e33764744c

I would have preferred #27981 to be merged instead, it was opened before and made by a new contributor. It's not that bad but not really nice for him. ;)

I don't mind =)

Cherry-picked for 3.1.2.


502 Bad Gateway from localhost port forwarding

I keep getting a 502 Bad Gateway when i try to run my project locally in Codespaces with port forwarding (port 5173).
I've tried killall node and creating a new instance of my project in codespaces.
I'm not sure what else to try.
The project is very simple to run:
npm install
npm run dev
https://github.com/CodeCrowCorp/cro-website-sv
Version: 1.73.1
Commit:<PHONE_NUMBER>f055b99068d3688932416f2346dd3b
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Embedder: codespaces

Resolved. needed to add --host to dev commands in package.
"dev:only": "vite dev --host"
Also added:
"forwardPorts": [5173]"
in devcontainer.json


Add tracing plugin test

Tracing was moved to plugin in PR #6001.
Add tracing related plugins to the plugin registration test.
Signed-off-by: Tony Fang<EMAIL_ADDRESS>

No, this is the only place to test plugin registration.

Minor nit: pulgin -> plugin in commit and PR title. Else LGTM.

@austinvazquez  Thanks for pointing out. Fixed both.

CI has error on CRI integration test on Windows platform not related to the change.
FAIL
mv: cannot remove 'C:/Windows/Temp/test-integration/containerd.log': Device or resource busy


Add devtool option to README sample config

Following from https://github.com/smooth-code/error-overlay-webpack-plugin/issues/8#issuecomment-404101038, I think this is a common enough issue that it's worth adding it to the README.
I'd be interested to know why this is the case so it can be added as a caveat.

Thanks!


Doctrine ORM 3 support

Feature Request
See https://github.com/sonata-project/SonataDoctrineORMAdminBundle/issues/1774

Seems like it's started by @franmomu in https://github.com/sonata-project/EntityAuditBundle/pull/590


Only initialize program builtins (2)

Only initialize program builtins
closes #1814
Description
Replace initialize_all_builtins for initialize_program_builtins
It also removes/changes tests that used the former function to comply with the new function
Checklist

[x] Linked to Github Issue

This PR was created to debug CI for pr #1815


feat(github-release): create a draft release

Added a "draft" option which defaults to false. If true, a draft release
is created (release.draft == true and no tag is created).tag_name and
target_commitish are still associated with the release for future tag
creation when draft is changed to false. Using the web UI this is
prepopulated on the release form. Via the API it uses the previously
stored values unless the call explicitly changes them.

waiting for https://github.com/googleapis/release-please/pull/702 to land before publishing this PR for review

waiting for https://github.com/googleapis/release-please/pull/702 to land before publishing this PR for review

@joeldodge79 went ahead and merged the snapshot update.

@joeldodge79 went ahead and merged the snapshot update.

_Perhaps the two prefixes we could use are pr-foo, when the option is specific to PRs, and release-foo when the option is specific to GitHub releases.

_Perhaps the two prefixes we could use are pr-foo, when the option is specific to PRs, and release-foo when the option is specific to GitHub releases.


Setting a field in config

Hi! Thank you for this nice gem. I have one enhancement proposal.
My use case is this:
We have a couple of microservices using Sidekiq. I want to track their performance in one series (say sidekiq_jobs, but I want to be able to indicate from which service it comes, so I can narrow down the metrics. Would it be possible to add something like this on config level?
For example:
chain.add Sidekiq::InfluxDB::ServerMiddleware,
     influxdb_client: InfluxDB::Client.new(options),
     series_name: 'sidekiq_jobs',
     additional_fields: { app: 'microservice_1' }

If you agree this is a good thing to add, I can try to work on it myself.

Great idea!
Can it be named just tags tho?
Can you do it and send a PR?


Edit Vundle installation instructions

Full GitHub URL is not required for Vundle (and vim-plug), as GitHub is automatically assumed.

Could you replace the paragraph from before with a new descriptive paragraph? I'd leave in a hyperlink to Vundle itself.

My bad, didn't see this earlier!
Hope that's alright. That was the style I've been seeing most often in other vim plugin READMEs. Should I still make another commit?

Yep, I just merged the commit with an added sentence. Thanks for the Vundle info!


Extract authInfo handling to reduce code duplication

Is there a related GitHub Issue?
https://github.com/cloudfoundry/cf-k8s-controllers/issues/291
What is this change about?
Extract handling the authorization info in API handlers into a dedicated wrapper. This has the consequence that the duplicated code (+related test code) around that is gone. There is sort of a drawback though - handlers' functions no longer comply to the http.HandlerFunc interface as they have the additional authInfo parameter.
To be honest, I can't really decide whether that change is worth it so I am looking forwards comments, suggestions and opinions.
Does this PR introduce a breaking change?
No
Acceptance Steps
Run the tests - they should pass

Merged this now in the interest of avoiding future merge conflicts


Run CI on all PRs.

I think we want PRs to be tested on all branches (such as beta branches). This is the default for Azure, so there's no need to specify the PR trigger.

r? @Eh2406
(rust_highfive has picked a reviewer for you, use r? to override)

@bors: r+

:pushpin: Commit 351f536ced02037cdb80b7319512468e0657f0ce has been approved by alexcrichton

:hourglass: Testing commit 351f536ced02037cdb80b7319512468e0657f0ce with merge 5e09ab589606befd16c73167462365bdec6eba54...

:sunny: Test successful - checks-azure
Approved by: alexcrichton
Pushing 5e09ab589606befd16c73167462365bdec6eba54 to master...


Revert "Update linguaggio.rst"

Reverts italia/design-doc#18: wrong branch.

Removing update from stable branch, moving to master.


Temp Table Delete issue

I, like others, have had similar issues where the temp tables aren't being deleted.  #60, #23 are along the same lines but in our case, we actually have deadlocks happening.
Situation:
We are first Bulk-Inserting 7-12 million rows into sql in the raw file formats using sql's bulk insert.  We then use Hangfire (another github project) to process batches of 50k with 5 concurrent jobs.  One of our c# calls is _context.BulkInsert(items, new BulkConfig() { PreserveInsertOrder = true, BulkCopyTimeout = 0, SetOutputIdentity = true});  This is the line that occasionally results in this error:

Hangfire has a built in re-try mechanism for failed jobs, so eventually this works and that bulk insert code does finish and run.  However, the jobs that create these (or any job that fails with the bulk insert call being made) leaves me with temp tables like this:

Proposed Solution:
Now with that being said, i'm not here to ask for a solution to the deadlock or any other exception caused by something unrelated to your library.  I also don't expect the bulk insert call to understand that it failed and clean up any previous attempts.  What I would like to see as a suggestion is another method on the context that I could call myself. Such as _context.CleanAndRemoveTempTables(); or something, that would run a process that does a clean up, but simply assumes all temp tables created by the same pattern as this library does today, can be removed, and does so.  Regardless of if there's data still in those temp tables or not.
Reason:
It would be impossible for the library to know when all temp tables should be done being used i suspect.  In my case, i'm even doing concurrent jobs across multiple instances of websites accessing the same DB.  Hangfire already allows me to run something 'after all other items are complete', but I don't have enough knowledge to understand how to 'clean up all temp tables that might have been created by bulk extensions library that are still here'.
Does that make sense?  Let me know what you think or if I can help provide more information.
Thanks!

A little more insight, the reason we chose to batch out in groups of 50k, is because we do a significant amount of processing from a few other tables before inserting them into the final table (which is the one we are calling _context.BulkInsert(...); for.  We had memory issues trying to do all 7-12 million in a list, so that was where we ended up.  I did see you have a batching process built into the library, but it just wasn't a single table that we needed to update with, so it didn't fit the needs of our items at the time.

Hae you tried setting UseTempDB_to true in BulkConfig?
That way Temp tables are created in temporary Db section and SqlServer would clean it up even after exception.

I'll try that tonight and let you know.

This seems to solve the problem, after adding the transaction around it, with the UseTempDb.   I re-ran it and got a few deadlocks and the temp tables where gone from both the DB i'm inserting and the tempdb one from the system tables.
Thanks.

Glad it solved the issue you had.


Add Shutdown and Close Functions for Graceful Connection Termination 2

Pull Request Template
Description:
this PR contains few fixes (including review comments) for this PR https://github.com/gofr-dev/gofr/pull/673 which seems to be abandoned but important for future improvements
Breaking Changes (if applicable):
No breaking changes to existing functionalities.

Additional Information:
Checklist:

[X] I have formatted my code using  goimport  and  golangci-lint.
[X] All new code is covered by unit tests.
[X] This PR does not decrease the overall code coverage.
[X] I have reviewed the code comments and documentation for clarity.

@iamgoroot, we also need to close the subscription manager, stopping all ongoing subscriptions. Stopping only the pubsub client won't work as it will continuously call the subscriber.
https://github.com/gofr-dev/gofr/blob/development/pkg/gofr/subscriber.go

@iamgoroot, we also need to close the subscription manager, stopping all ongoing subscriptions. Stopping only the pubsub client won't work as it will continuously call the subscriber.
https://github.com/gofr-dev/gofr/blob/development/pkg/gofr/subscriber.go

yeah I know and already have some fixes locally. Will commit later

I'm still wondering what you try to achieve with the cancelable context in startSubscriber. Because for me, the current implementation doesn't need a cancelable context. I would have understood a timeout but not a cancelable context while there is no go routine involved.
But, the code is way better now than it was.
So I would say go for it.

It may look that way. But there's dependencies and they have code too. Did you have a chance to look deeper into those? For example Kafka client implementation somewhere here
https://github.com/segmentio/kafka-go/blob/7b9c99dd6562f80bd3fc447305115029a7154382/reader.go#L793
Anyway I'm not interested in supporting this anymore. It has signs of merge/explain loop for feature users of opinionated framework have no use for

I'm sorry if my comments about your PRs led to closing it.
Your PR was great, and I think you were very close to have something production ready.
But I also understand that a PR with numerous comments and questions about the implementation on almost a week is a pain.


Document update

Please review @roywant @schilit

@roywant are you done reviewing?

@g-ortuno We can’t automatically merge this pull request. please sync

Merged with master


Preset of ajax result

Hi, I tried your library but i have a little problem : how may I preset a data in my ajax data list, only with the Id ? I've read in the documentation that we have to put the 'initSelection' method but if it's the solution, i don't understant how...
Thank you for your help

Sorry, I'm trying to wrap my head around what it is you're exactly trying to do.... So you're using the ajax module if I understand correctly, with the ajax option for configuring your requests, right? Maybe you could use the query option? It's a bit more cumbersome because you need to the AJAX request yourself, but it will allow you the most freedom.

Closing due to inactivity...


passthrough config if there are no secrets defined (address #204559)

Description of changes
In the case where sharedFolders is an empty list, simply copy through the config file. Addresses #204559.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[x] tested module with sharedFolders = []
[x] Fits CONTRIBUTING.md.

@jwoudenberg, @RyanGibb, @thoughtpolice, @dali99
were on the original PR.

This looks good to me!

Can we please merge either this or #205633 ?


Should WAL Country code  be permitted as a county of birth.

@geoffj-FUG
While experimenting with the VLD downloader I have observed that files containing Country Codes for both Verbatim and Alternate County of Birth of WAL.
Clearly CEN1/2 permits this. CSVProc has this as an error.
Do we continue to error it? Do we change CSVProc to accept it or produce a warning rather than an error?
It is not a valid chapman code

Kirk
We shouldn’t be using Country codes. There is a standard list but we use only OVF and OVB.
I asked that question when we started CSVProc and the answer was no change.
I suggest that they might have been early FC1 pieces when they would still have been experimenting.
Geoff
From: Kirk Dawson<EMAIL_ADDRESS>Sent: Friday, December 11, 2020 11:31 AM
To: FreeUKGen/FreeCENMigration<EMAIL_ADDRESS>Cc: geoffj-FUG<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: [FreeUKGen/FreeCENMigration] Should Country codes be permitted as a county of birth. (#1059)
@geoffj-FUG https://github.com/geoffj-FUG
While experimenting with the VLD downloader I have observed that files containing Country Codes for both Verbatim and Alternate County of Birth.
Clearly CEN1/2 permits this. CSVProc has this as an error.
Do we continue to error it? Do we change CSVProc to accept it or produce a warning rather than an error?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/FreeUKGen/FreeCENMigration/issues/1059 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AKCPIFLKCRWG7I3V32QEEQDSUFY6VANCNFSM4UV3WUJQ .  https://github.com/notifications/beacon/AKCPIFNZ6KIEKANMF4UL5ILSUFY6VA5CNFSM4UV3WUJ2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4LLHXXAQ.gif

Issue is the use of WAL as a chapman code rather than WLS

WAL used to be rejected by either CSVCheck or Valdrev, (I am not sure which, maybe both). I have corrected it many times when it has cropped up when the piece was tested. So it has been accepted by a validator along the way even though there was a Warning against it. WAL is not in the original list of valid Chapman Codes from the early 2000s. I copied and pasted that original list from the documentation to get my list of Chapman Codes.

Quite a few of the Welsh Census often say just Wales and no County.
Probably because of the spelling of many POB’s.
The enumerator went for the easiest.
CSVCHECK accepts WLS but rejects WAL as Geoff says.
On Sat, 19 Dec 2020 at 10:22, geoffj-FUG<EMAIL_ADDRESS>wrote:

WAL used to be rejected by either CSVCheck or Valdrev, (I am not sure
which, maybe both). I have corrected it many times when it has cropped up
when the piece was tested. So it has been accepted by a validator along the
way even though there was a Warning against it. WAL is not in the original
list of valid Chapman Codes from the early 2000s. I copied and pasted that
original list from the documentation to get my list of Chapman Codes.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/FreeUKGen/FreeCENMigration/issues/1059#issuecomment-748453973,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADIL3VXR7FI7ENRV4L4JWFTSVR5HPANCNFSM4UV3WUJQ
.

@FreecenBren @geoffj-FUG I am sorry to have to tell you that RG121953.VLD contains WAL as a Chapman Code on CEN2 so something is not right somewhere

Kirk
I immediately know who transcribed that. I used to do a replace all on his spreadsheets. I must have missed one. Not sure why it validated though.
Mea Culpa
Geoff

No problem; just needed to ensure we were not missing something; looks like Valdrev does not check.
Will close


Add use of pointer_pose into the demo

I was almost at the point of hacking in some pointer based on hand tracking, with filtering to take account of the noise in the position of the hand.
But then it seems there is already this function in the API: https://developer.oculus.com/documentation/native/android/mobile-hand-tracking/

PointerPose
Deriving a stable pointing direction from a tracked hand is a non-trivial task involving filtering, gesture detection, and other factors. The hand tracking API provides the PointerPose field on the ovrInputStateHand structure so that pointing interactions can be consistent across apps.
The PointerPose field is an ovrPosef that indicates the starting point and position of the pointing ray in world space. It is recommended that developers use this field to determine the direction the user is pointing for the case of UI interactions.

And then it seems that it's already been pulled into the oculus mobile here:
https://github.com/GodotVR/godot_oculus_mobile/pull/83
Use of this function should be included in the demo examples, because that's as good as what we use for documentation of anything.  Otherwise people are going to not know it's there and waste time implementing their own version, as I almost did.

I know this is not as useful as a finished pull-request to the library, but I am not confident enough to make one.
Here is my development at working out the pointer_pose function, as well as the fade-out and fade-in feature you get on the hand-tracking when the confidence level drops and you are in the default oculus system scene.
I found that it is best to put the hand model and pose laser pointer into a Spatial node that is a sibling of the ARVRController instead of as a child of the controller node.  This is for two reasons:
First, the laser pointer transform is given relative to the ARVROrigin, not the ARVRController, because it appears really to work as 3DOF where it is close to the vector from the ARVRCamera to the CofG of the ARVRController/hand so that it does not deflect when you make, for example, a pinch gesture.
Second, during the fade-out when the confidence level drops you don't want the hand to continue moving, so being able to stop copying transform over from the ARVRController node to its sibling instead of trying to suppress the parent transform is a lot smoother to implement.
It turns out you have to have the confidence level high if you are using the hands for controlling a UI rather than playing a rubber hand game because otherwise you get a lot of mis-clicks and actions made that the player did not intend.  It is much worse to click on a button by mistake than to fail to click on a button that you intended to.  By disappearing the hands whenever the confidence drops (most commonly when they occlude one another) the player can very quickly be trained to keep their hands apart when doing work, which really improves the experience.
HandScript.zip


Generate method body for the delegate Invoke method

We didn't really need this before because RyuJIT understands Invoke
method calls and inlines them. This is only to cover the case of ldftn
of the Invoke method and reflection-invoking the invoke method.
We could possibly also implement the body as this.Invoke(args...) (and
have RyuJIT expand that) but this has the benefit of also supporting
codegens that don't expand Invoke. On the other hand this has the
drawback of emitting a calli that will get the (unnecessary) fat
function pointer treatment.

Looks good to me.


Change distribution method of the module

The old method of distribution was inconsistent with different platforms.
I've now changed it to use the standard "console_scripts" way. This simplifies a lot of the installation process on windows machines.
This should solve issue #8 once and for all.

I believe this is a great time to bump the version number to 0.1.7


Step2 Source server type is wrong

For Source server type, Contoso selects SQL Server on Azure Virtual Machines.

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

ID: 99fa4eb8-fc61-25c9-1e8d-f52a0eac1381
Version Independent ID: 7958b3aa-d184-5f73-7364-6f99913614a9
Content: Assess on-premises workloads for migration to Azure - Microsoft Cloud Adoption Framework for Azure
Content Source: docs/cloud-adoption/migrate/azure-best-practices/contoso-migration-assessment.md
Service: site-recovery
GitHub Login: @BrianBlanchard
Microsoft Alias: brblanch

@cengxiye Thank you for your feedback! Can you please provide more details around this issue for better clarity.

@cengxiye Just realize what you are mentioning. I will create a PR to fix this. It should be "For Source server type, Contoso selects SQL Server and for Target server type, Contoso selects SQL Server on Azure Virtual Machines.

@cengxiye Thank for the catch! We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


quickbooks-desktop.rb: added :no_check comment

cc @adityadalal924.

Merged as 83addcf3784077ddad4b7a30ffb7db02f4c18d5e.

Would be a nice thing to also add the the new rubocop cask rules https://github.com/caskroom/homebrew-cask/pull/15925

@adityadalal924 Feel free to open an issue over on jawshooah/rubocop-cask, or hack it out and submit a PR.
I'd be willing to transfer ownership to the caskroom organization if anyone else wants to help extend and maintain it.


Questions about the results

Hello, I try to evaluate the uploaded trained restaurant model by running evaluation.py directly, did not make any changes to the code, but did not get the same results as the paper, is it necessary to tune some parameters? Thank you for your answer.
That's the result I got.
--- Results on restaurant domain ---
precision    recall  f1-score   support
     Food      0.654     0.493     0.562       887
    Staff      0.406     0.270     0.324       352
 Ambience      0.245     0.143     0.181       251
Anecdotes      0.000     0.000     0.000         0
    Price      0.000     0.000     0.000         0

Miscellaneous      0.000     0.000     0.000         0
avg / total      0.527     0.381     0.442      1490

All the parameters are already set correctly as default and there is no need to tune. If you follow the evaluation instruction in README, the results should be exactly as follows (I just checked again by downloading the current code):

Did you uncomment line 28 in evaluation.py to set the model to be evaluated as the pre-trained model ?

Yes，i  uncomment line 28 in evaluation.py to set the model to be evaluated as the pre-trained model,and i just change the import lib in model.py as follow：
import logging
import os
os.environ['KERAS_BACKEND']='theano'
import keras.backend as K
K.set_image_dim_ordering('th')
import importlib
importlib.reload(K)
from keras.layers import Dense, Activation, Embedding, Input
from keras.models import Model
from my_layers import Attention, Average, WeightedSum, WeightedAspectEmb, MaxMargin
Shouldn't I be doing this?This is the result of my downloading and running again.
precision    recall  f1-score   support
     Food      0.855     0.729     0.787       887
    Staff      0.792     0.636     0.706       352
 Ambience      0.781     0.470     0.587       251
Anecdotes      0.000     0.000     0.000         0
    Price      0.000     0.000     0.000         0

Miscellaneous      0.000     0.000     0.000         0
avg / total      0.827     0.664     0.734      1490
thanks a lot!

Seems you are using Python 3. The code is tested under Python 2.7. Not sure whether this will affect the results. Also check the versions of other dependencies.

Did you download the preprocessed datasets directly or you preprocessed the original datasets again using the script provided? You should use the preprocessed dataset for the saved model. If you preprocess the dataset again, the word indexes may not exactly match the saved word embeddings as I don't really remember whether I changed something in the preprocess script when cleaning up the code.

thanks!i try to use  Python 2.7, and get a similar result

I also tried to replicate the results and I got the same as @ruidan. It is a close result to the paper, but not exactly. Was it a different training set or parameters used in the paper?

@ThiagoSousa thank you!


⚠ CIOOS Pacific ERDDAP dataset DFO_MEDS_BUOYS has degraded performance

In 6dab55e, CIOOS Pacific ERDDAP dataset DFO_MEDS_BUOYS (https://data.cioospacific.ca/erddap/tabledap/DFO_MEDS_BUOYS.htmlTable?STN_ID&time>=now-1week&distinct()) experienced degraded performance:

HTTP code: 503
Response time: 14596 ms

Resolved: CIOOS Pacific ERDDAP dataset DFO_MEDS_BUOYS performance has improved in 3463613 after 6 minutes.


Getting DB error 401

HI there
I have mantis bt running just fine on a Windows server. Using postgresql as the db
Now I am trying to load up some plugins like your announce, and I am getting this error
APPLICATION ERROR #401
Database query failed. Error received from database was #-2: ERROR: syntax error at or near "d"
LINE 1: DELETE d.*
^ for the query: DELETE d.*
FROM BT_plugin_Announce_dismissed_table d
LEFT JOIN BT_plugin_Announce_context_table c on c.id = d.context_id
WHERE c.id IS NULL.
Please use the "Back" button in your web browser to return to the previous page. There you can correct whatever problems were identified in this error or select another action. You can also click an option from the menu bar to go directly to a new section.
Do you know how I can fix this and get your plugin to work for me?

Resource interpreted as Stylesheet but transferred with MIME type text/html: "http://<IP_ADDRESS>/bt/plugin_file.php?file=Announce/announce.css".
plugin_file.php?file=Announce/announce.js:1
Also when I looked in the browser F12 tools, I saw this error.
ALso when I tried to get a different plugin working Taskodrome, I think it gave me similar errors like this in the F12 tools
I am using version 2.15.0 of the mantis bt software, and I am trying to use your older version 2.11.  It installed this time, but now is giving these other others.
I was able to create an announcement, but up at the top of the page where it should be at, there is just like some whitespace all the way across the top of the page.  And that is where the message is supposed to be at.
So I think maybe the reason your latest version didnt work, is because I am using version 2.15.0 and I think your newest plugin needs the newest version of mantis.  Which I may not be installing, since I have everything else working perfectly on version 2.15
Well, I am hoping you can help me with this issue, and at least get version working in my environment. Everything else in my mantis bt is running OK as far as I know.   Just not these plugins :-O

I created #42 to track your second issue, which is not related to the original post.

@gamerboy11 I could reproduce the SQL error in a fresh install of Mantis 2.17.1 and Announce plugin 2.3.0. Will look into it.

@gamerboy11 I have a fix nearly ready for this, just pending some tests. I should be able to push it later today if all goes well.
Until then, since this is a new install, you should be able to workaround the problem by

installing version 2.1.1, then
runupdate mantis_config_table set value=5 where config_id = 'plugin_Announce_schema';

This will bypass the upgrade step which contains the offending SQL.
you can then upgrade the plugin to the latest version.

Of course you might still get errors if you delete an announcement or perform some other operation that triggers a deletion from the dismissals table (there is a similar SQL statement in dismiss.php).


[ExplicitModuleBuild] Add a dependency scanning test for package interface

Add a test for dependency scanning for package interface to ensure the correct interface is found when package interface is enabled.

@swift-ci please smoke test


ensure correct site_id is always returned in select * from crsql_changes

added a 3 databases round-robin sync to the e2e test case where we confirm site ids after sync.

@alexerrant -- this might be useful to you given it fixes a problem around blobs in the wasm build of sqlite.
-- https://github.com/vlcn-io/wa-sqlite/commit/d90673906f01f251128abb25fde491df3e0aceac


(346) 779-7975

(346) 779-7975
Originally posted by @bejaranobasilia45 in https://github.com/bejaranobasilia45/3467797975/issues/7#issuecomment-570816936(346) 779-7975 #runSETUP

Www.trackMyip.cpm<IP_ADDRESS>

<IP_ADDRESS>

Www.trackMyip.com<IP_ADDRESS>

Www.trackMyip.com<IP_ADDRESS>

Www.trackMyip.cpm<IP_ADDRESS>


Add cached backend

Fixes #20.
Adds a backend cache:
from grand.backends import SQLBackend, InMemoryCachedBackend
from grand import Graph

G = Graph(backend=InMemoryCachedBackend(SQLBackend(...)))

G.nx.degree(42) # May be slow if the graph is very large
G.nx.degree(42) # Will be very fast, since the result is cached
G.backend.clear_cache() # Clear the cache explicitly

Codecov Report

Merging #28 (52721e7) into master (ee89e73) will increase coverage by 3.89%.
The diff coverage is 98.00%.

@@            Coverage Diff             @@
##           master      #28      +/-   ##
==========================================
+ Coverage   51.59%   55.48%   +3.89%
==========================================
  Files          15       16       +1
  Lines        1068     1166      +98
==========================================
+ Hits          551      647      +96
- Misses        517      519       +2

Impacted Files
Coverage Δ

setup.py
0.00% <ø> (ø)

grand/backends/backend.py
80.89% <95.00%> (+10.89%)
:arrow_up:

grand/backends/__init__.py
86.66% <100.00%> (ø)

grand/backends/test_cached_backend.py
100.00% <100.00%> (ø)

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 ee89e73...52721e7. Read the comment docs.


Include ListComplete method in mock interfaces

Update comment for client constructors.

Fixes:
https://github.com/Azure/azure-sdk-for-go/issues/6559
https://github.com/Azure/azure-sdk-for-go/issues/6605

CC @ArcturusZhang

🤖 AutoRest automatic publish job 🤖
success (version: 2.1.140)


⚠ Cioos Pacific CKAN Sitemap has degraded performance

In ba93347, Cioos Pacific CKAN Sitemap (https://catalogue.cioospacific.ca/sitemap/sitemap-1.xml) experienced degraded performance:

HTTP code: 200
Response time: 9302 ms

Resolved: Cioos Pacific CKAN Sitemap performance has improved in ce5d670 after 6 minutes.


Custom routes

Is your feature request related to a problem? Please describe.
For SEO reasons my routes need to be in a specific language, the react components are in English so this is a problem if i cannot change them to for example Swedish. I found this older issue with similar problem but that does not work: https://github.com/danielcondemarin/serverless-next.js/issues/98, looking at the documentation we cannot specify routes
Describe the solution you'd like
Next has a plugin called next-routes where you can specify routes like this:
module.exports = routes()     .add('home', '/hem', 'home')     .add('about', '/om-oss', 'about')
If we can use this plugin it will solve this issue.
Also i have checked the manifest.json file, if we can modify it somehow before deployment that could work too.

@Xocix This component uses next's public filesystem routing.
I suggest you raise the issue in next's repo, right now this is outside of the scope of this project.


Extract code snippets from how-to-delete-all-the-comments-by-an-author-from-all-the-slides-in-a-presentatio.md

Extract the code snippets from docs/presentation/how-to-delete-all-the-comments-by-an-author-from-all-the-slides-in-a-presentatio.md and use docfx snippets to retrieve them from the samples folder and audit the text.

closing as duplicate


[Doc] Rollups enhancements

See https://github.com/opensearch-project/index-management/issues/408

Feature Review is 7/22

This meta issue contains 4 sub-issues:

doc_count from source index -> in 2.2
allow dynamic target index in rollup searches -> in 2.2
allow rollup searches to multiple rollup indices -> in 2.2
allow query_string queries -> pushed to 2.3.

Issues #1-3 are documented. Waiting on a deployment to test these. Waiting for tech review in the meantime.

We'll  make a new issue for item 4 on the @kolchfa-aws's list. Closing for now.


Use jest-light-runner for tests

Jest monkeys around with NodeJS's environment and module loader in various ways.
Because parts of Livepack are quite low-level, this makes it hard to be confident from the tests that it's operating correctly, related to globals and module loading.
jest-light-runner is a Jest runner which takes a more lightweight approach. It runs each test file in a NodeJS worker thread, rather than using vm module. It makes less changes to the environment.
jest-light-runner also supports dynamic import() so could:

Run all tests in Jest, rather than using Mocha for splitAsync tests
Remove the shim for import() which is used in tests in lib/serialize/createModule.js

The commit above switches the test runner to jest-light-runner.
It also creates fixtures as virtual files rather than writing them to the actual filesystem. It patches fs and Module to do so.
There remains one kink: When running splitAsync.test.js in the main jest run, it causes unexplained test failures in other test files which run in the same worker thread (I fiddled with the code in piscina, which jest-light-runner uses for its thread pool, to verify that the test failures were always in test files run in the same worker thread as splitAsync.test.js).
The errors are so weird, I actually suspect a bug in NodeJS / V8.
Demonstration of this problem on jest-light-runner branch.

34fbcd86daec3b78961046dba6537f8ffdd2c8ad fixes the splitAsync.test.js kink by sequencing tests so splitAsync.test.js test file runs last.


[sveltekit-pod] 4. Add the kit to CLI run command

Background
At this point, the kit should be fully established and ready for an individual to use via the CLI.
Acceptance

[x] Add the kit to the CLI as a setup option
[ ] Test it locally with a fresh install

Changing version number from 0.1.0 to 0.0.1 similar to other kits.


NF: UML Elements

Achimate as a notation does not provide sequence diagrams nor state machine diagrams.
I am aware that it is possible to emulate them using some elements from Archimete but this is ugly and not really compliant not with Archimate nor with UML.
How about allowing drawing some of UML diagrams as Skatch diagram is allowed. This would help a lot when we want to have all types of diagrams in the single repository. I am concerned especially on sequences (this would be highest priority) and next state machine diagrams (as nice to have). All the rest UML is easy to mimic in Archimate.

I don't think this is likely in the immediate future. Unless we completely refactor the whole of Archi's Figure framework to allow different types of shapes (and more shapes).


support next 10

Hello! This package is really useful, I've been using it for routing on a static build of a next project and it's worked beautifully.
Unfortunately there's been an issue with conflicting peer deps using npm 7, which complains that because some other deps require next ^9.0.0 || ^10.0.0, and others require >=7.0.0,
next-static-manifest specifically requires ^9.5.0, and my project is using next 10.1.3.
I used the  npm install --legacy-peer-deps to work around it, but ideally this would be fixed here instead.
(https://docs.npmjs.com/cli/v7/commands/npm-install#configuration-options-affecting-dependency-resolution-and-tree-design
https://twitter.com/wesbos/status/1359173446740172810?lang=en
https://github.com/npm/rfcs/blob/latest/implemented/0025-install-peer-deps.md
)
I've gone for the conservative option here, but happy to change it to >= 9.5.0 if needs be.

Thanks @willheslam! Happy this approached worked for you as well.
PR should be live in the latest version<EMAIL_ADDRESS>


Fix link to Facebook config management paper.

Signed-off-by: James Peach<EMAIL_ADDRESS>

Thank you @jpeach


Leetcode Question Number 1448

Leetcode Question Number 1448 solution needs to be added in the repo .
The link for the question is this.
Accepted Languages C++ / JAVA .
You Only need to add the  class Solution in the file.
Make Sure you have read the contributions guidelines before creating an PR.

Please assign


Implement RecordBatch::concat

Which issue does this PR close?
Closes #461 .
Rationale for this change
As described in the issue, tries to implement RecordBatch::concat according to https://github.com/apache/arrow-datafusion/blob/master/datafusion/src/physical_plan/coalesce_batches.rs#L232 .
What changes are included in this PR?
Adds a new function concat to struct RecordBatch, and test cases.
Are there any user-facing changes?
With this fix, a new RecordBatch could be created by concatenating multiple RecordBatches.

Codecov Report

Merging #537 (e1f24d1) into master (f1fb2b1) will increase coverage by 0.01%.
The diff coverage is 91.42%.

:exclamation: Current head e1f24d1 differs from pull request most recent head eec5c90. Consider uploading reports for the commit eec5c90 to get more accurate results

@@            Coverage Diff             @@
##           master     #537      +/-   ##
==========================================
+ Coverage   82.60%   82.61%   +0.01%
==========================================
  Files         167      167
  Lines       45984    46054      +70
==========================================
+ Hits        37984    38048      +64
- Misses       8000     8006       +6

Impacted Files
Coverage Δ

arrow/src/record_batch.rs
90.14% <91.42%> (+0.88%)
:arrow_up:

parquet/src/encodings/encoding.rs
94.85% <0.00%> (-0.20%)
:arrow_down:

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 f1fb2b1...eec5c90. Read the comment docs.

Although @nevi-me seems to like RecordBatch::concat -- https://github.com/apache/arrow-rs/issues/461#issuecomment-877784891

Hi @alamb, refer to @jorgecarleitao latest comment #461 (comment), it seems that arrow::compute::concat::concat_batches may be better (even not good enough for thread model).
I will follow the discussion and try to update this PR.

thanks @silathdiir !

Unless I hear different, I plan to merge this PR tomorrow (and include it in 5.0.0)

Thanks again @silathdiir !


fixing with ast literal and generate hash in app for dashboard

Pull Request Test Coverage Report for Build 1210

5 of 6 (83.33%)  changed or added relevant lines in 2 files are covered.
52 unchanged lines in 15 files lost coverage.
Overall coverage increased (+0.05%) to 78.811%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

datmo/dashboard/app.py
0
1
0.0%

Files with Coverage Reduction
New Missed Lines
%

datmo/cli/command/base.py
1
93.98%

datmo/snapshot.py
1
83.49%

datmo/cli/command/run.py
2
81.5%

datmo/core/util/i18n.py
2
100.0%

datmo/cli/command/session.py
2
95.7%

datmo/logger.py
2
92.59%

datmo/cli/command/snapshot.py
2
100.0%

datmo/cli/command/datmo_command.py
2
100.0%

datmo/core/controller/file/driver/local.py
2
91.95%

datmo/core/controller/code/driver/file.py
2
94.12%

Totals

Change from base Build 1208:
0.05%

Covered Lines:
4426

Relevant Lines:
5616

💛  - Coveralls


[Inbox] markdown-to-jsx is affected by 7.1.7

The current version<EMAIL_ADDRESS>is affected by Cross-Site Scripting
see https://security.snyk.io/vuln/SNYK-JS-MARKDOWNTOJSX-6258886 for more details.
How to fix?
Upgrade markdown-to-jsx to version 7.4.0 or higher. I found this commit https://github.com/trycourier/courier-react/commit/e9f89db6a1d75255aeffcce64154d31ca70b198b so it will probably require some extra work and not just a version bump.

yeah there was an issue with a customer where their app was breaking because they got a newer version. let me poke arond and see what their issue was


Add Licence File

Hi @jeppe742  Would you mind if I make a PR copying the exact Delta-rs license file? I need the license in order to use this library internally in our firm.
Thanks!

Sounds like a great idea. Thanks!


Custom user store with IdentityServer4

Hello,
There are many examples how to write custom user store service in IdentityServer3 (for instance:
https://gist.github.com/tjrobinson/0ad6c790e90d7a385eb1, https://github.com/IdentityServer/IdentityServer3.Samples/tree/master/source) but I can't find any tips how to implement this in IdentityServer4.
My target is to authenticate user in a number of Active Directiories.
One idea is to use Aspnet Identity and write own Membership Provider but I think there should be simplier solutions.
Thank you very much for your help,
Maciek

That's a main difference in IS3 and IS4 -- in IS4 that's entirely up to you UI code.

Thank you,
I understand that in web scenario I should implement something like example AccountController (https://github.com/IdentityServer/IdentityServer4.Quickstart.UI/tree/release/Quickstart/Account).
My case is other.
I implement mobile apps (one for android and one for ipad). In my mobile apps I want to use my API in REST and protect them with authentication and I want every user works in own context (authenticates with own username/password). Also I don't want authentication in browser (mobile app->browser->mobile app). I need input login and password in mobile app.
I'm just starting to deal with authentication with OpenId Connect. I've read lots articles. All of these say that OpenId Connect abstracts from how server authenticates user but I still can't imagine what should be flow for mobile/native application.
Do I think correctly?

User inputs username/password into mobile app
Mobile app sends with POST method AuthN Request together with username and password data (over SSL) to IdentityServer
IdentityServer validates username and password. How to add custom validation over my own user stores?
IdentityServer sends response with ID Token
ID Token will be stored on device and use to authenticate in services (until token expires)

ps.
Sorry for my english..

That's essentially the resource owner password flow.

Thank you for the answer.
In documentation (https://identityserver4.readthedocs.io/en/release/quickstarts/2_resource_owner_passwords.html) is written:
The spec recommends using the resource owner password grant only for “trusted” (or legacy) applications. Generally speaking you are typically far better off using one of the interactive OpenID Connect flows when you want to authenticate a user and request access tokens.
I understand, that if mobile app is our implementation than we can treat it as "trusted" application?
Is there any other possibility for non-browser authentication with OpenId Connect except ResourceOwnerPassword?

That's the only user-based non-browser login flow. The browser offers lots of additional security features, and that's why it's recommended.

Thank a lot for claryfing!
So, I will use OpenAuth resource owner grant.

What are interfaces that we have to implement for custom user store if we only have to implement Implicit flow?

I have found something like that https://damienbod.com/2017/04/14/asp-net-core-identityserver4-resource-owner-password-flow-with-custom-userrepository/
Maybe it will help

@m-shanti I was wandering if you have seen this url:
There are some code samples there that could help you out

I am also having an issue implementing the custom user store/authentication that uses the authorization code flow.
Most of the example I found seems to suggest implementing the IResourceOwnerPasswordValidator which doesn't seem to be called for authorization code flow.
Is there another interface that I need to use instead for the authorization code flow? Any info/link to an example would be much appreciated. Thanks!

Maybe https://mcguirev10.com/2018/01/02/identityserver4-without-entity-framework.html (code at https://github.com/MV10/IdentityServer4.AdoPersistence) by @MV10 and https://www.scottbrady91.com/Identity-Server/Creating-Your-Own-IdentityServer4-Storage-Library .
There are probably others, but these seem to work with quickly trying. If someone does something similar and can share a link to a repo, would nice, naturally.

thanks @veikkoeeva for your comment. I've gone through all of those links (and more!) and none seems to fit my scenario which seems pretty basic. All I needed to do is to get authorization code flow working with a custom user store .
Good news though! I finally managed to figure this out. The problem was that I was using the QuickStart IdentityServer template provided and after looking at the AccountController source code by default it is only validating the users inside TestUserStore.
The solution is to simply replace the TestUserStore provided with my own authentication implementation based on an interface (e.g. IAuthenticateUserCommand) that performs user validation against my user database. I also then implemented the IProfileService for retrieving the user claims (e.g. role, etc).


nzSelectedKeysChange event handle of tree component don't work!

Version
7.0.0-rc.1
Environment
stackblitz
Reproduction link
https://kpwdbd.run.stackblitz.io
Steps to reproduce
click some node of the tree, the function who's name is nzSelect should be call. then should have some message be print to the console, but only click handle be call,  nzSelectedKeysChange handle not work!
What is expected?
nzSelect should be call!
What is actually happening?
It's not work!
Other?
除了nzSelectedKeysChange的回调没有执行，双向绑定nzSelectedKeys的defaultSelectedKeys也没有改变

@duanoldfive will fix it, nzSelectedKeys not support ngModel yet(doc will be updated until we fix it)

@simplejason thank you for your answer!

ref #2628


Adding example for inject-shellcode-using-mapping-injection.yml

Example file for inject-shellcode-using-mapping-injection.yml - https://github.com/mandiant/capa-rules/pull/513

thanks!


Update OWNERS file

This PR:

Updates OWNERS with a current and more limited list of approvers and reviewers.

@hanjiayao 👋 Questions for you:

Who on your team would you like to have reviewer/approver status?
Who on your team would you like to have repo admin permissions?

/assign @chenopis

@spiffxp 👋 In both this PR and https://github.com/kubernetes/kubernetes-docs-zh/pull/497#issuecomment-414428304, despite @Bradamant3 and myself being approvers in kubernetes-docs-zh/OWNERS, k8s-ci-robot seems sluggish in either adding an approval or an error message. Any suggestions? Otherwise, I can use my admin powers to manually merge. Thanks in advance! 🙇

@zacharysarah I'm willing to bet the approve plugin isn't turned on for this repo, ref: https://github.com/kubernetes/test-infra/issues/6227
If you'd like to, you can PR this into k/test-infra yourself.  You can also open an issue and assign to me and I'll move forward on it without seeking lazy consensus, given that the request is coming from someone in the OWNERS file.

@spiffxp Thanks! Done.

ping @chenopis 👋

IMO, i'm Chinese, own in cn repo good for me yet.  BTW, due to contribute in en docs and kubernetes core repo, i apply for to be approver, if not mind?

@Bradamant3 👋 Would you mind tagging this again with a /lgtm, please?

@stewart-yu 👋

i apply for to be approver, if not mind?

That sounds great to me. Please work with @hanjiayao to see what the team's needs are!
/cc @tengqm


cols in textarea

Any chance of adding a cols attribute option to the textarea?

how that is related to x-editable?
you can do that using html  or using javascript also right.

Sorry, was not particularly clear with my comment.
On lines 3025 and 3043 here: https://github.com/vitalets/x-editable/blob/master/dist/bootstrap3-editable/js/bootstrap-editable.js it shows that I can enter a rows attribute when using a textarea as the editing input.
I was just wondering whether you could add a cols attribute too for more control over the size of the textarea?

I get you now,
but unfortunately we don't have such option.
if you use bootstrap, you can restict with col-*

ok thanks!


New: Adding @afterScenario event to reset DB

Done the changes suggested.

Looks good, can you merge in develop so I can merge


feat(@angular-devkit/build-angular): set document locale with i18n

When using i18n (i18nLocale), set the document locale in index.html using the html lang attribute.
Fixes #8102

Thank for the contribution.  This was something that we were interesting in including for 9.0.
An exception can be added for the lint failure via an inline comment.  That function will be refactored in the medium-term anyway.
The other CI failure is a flake and can be ignored.


Clean up untriggerable asserts

In the context of testing/fuzzing, we may want every assert to be triggerable by a state transition test (this is arguably part of "100% test coverage").
There are currently 53 asserts in 0_beacon-chain.md. The following 9 seem untriggerable by a bad block. We should consider removing them (e.g. changing them to sys.exit(1)) to clarify that those assertions are for sanity checking and/or documentation and are not expected to trigger.
# get_permuted_index
assert index < list_size
assert list_size <= 2**40

# get_crosslink_committees_at_slot
assert previous_epoch <= epoch <= next_epoch

# get_block_root
assert slot < state.slot <= slot + SLOTS_PER_HISTORICAL_ROOT

# get_state_root
assert slot < state.slot <= slot + SLOTS_PER_HISTORICAL_ROOT

# get_randao_mix
assert get_current_epoch(state) - LATEST_RANDAO_MIXES_LENGTH < epoch <= get_current_epoch(state)

# get_active_index_root
assert get_current_epoch(state) - LATEST_ACTIVE_INDEX_ROOTS_LENGTH + ACTIVATION_EXIT_DELAY < epoch <= get_current_epoch(state) + ACTIVATION_EXIT_DELAY

# get_beacon_proposer_index
assert previous_epoch <= epoch <= next_epoch

# integer_squareroot
assert n >= 0

many of these are for usage outside of the state transition function. If someone wanted to run get_block_root against a slot/state, they need to have some sort of bounds checking.
For example clients run get_crosslink_committees_at_slot often locally to figure out assignments for the current and next epoch. They should have explicit guidance in the def of the function to show the bounds so that they know to handle the exception when they try to do something like get assignments for two epochs in advance.

I'm not sure what is the most Pythonic way to discriminate "spec code" from "implementer guidance". In the Go executable spec the distinction is made between return err (for expected error handling in the spec) and panic(err) (for implementer guidance that is not expected to trigger in the context of the spec).

@JustinDrake A Pythonic way is that we can define a custom exception for the spec error; in Py-EVM there's a ValidationError (class ValidationError(Exception): pass), and we can raise ValidationError explicitly.
I think we did consider it but also worried about that might be too Pythonic compare to simple assert for the readers.
@djrtwo What do you think about it?

assert lgtm as an explicit way to document pre-conditions and valid argument values in a spec - there are similar hidden pythonisms (such as array-out-of-bounds accesses etc) that are best brought to light this way, I believe.
one problem we've had in the past with these assertions was that they were expressed in terms of a block-transition-in-progress - that has substantially improved with the shifted epoch processing, but I would be keen to see / know if there are any exceptions to that rule, where certain functions can only be used at certain points of the transition function.

We're down to just three untriggerable asserts 🎉 (#1237 removes the assert in integer_squareroot):
# shuffle_index
assert index < list_size
assert list_size <= 2**40

# get_block_root_at_slot
assert slot < state.slot <= slot + SLOTS_PER_HISTORICAL_ROOT

Down to two:
# compute_shuffled_index
assert index < index_count

# get_block_root_at_slot
assert slot < state.slot <= slot + SLOTS_PER_HISTORICAL_ROOT


Add controlled unitary higher-level operators

Added CNU, Cascade already defined.

CU, CascadeU defined

reverseCascade, reverseCascadeU, and collect/expand have been added, as well as simU.


Activation offloading for fullfinetuning + fix tied embedding

Context
What is the purpose of this PR? Is it to

[x] add a new feature
[x] fix a bug
[ ] update tests and/or documentation
[ ] other (please add here)

Changelog

Added activation offloading for single/distributed full finetuning
Found that we didnt address the tied embedding issue. Fixed it by adding a dummy nn.Module so that tensor hooks would work with it
Moved the context manager logic to a function
Made the recipes more aligned

Test plan

Will add the option to all the configs for full finetuning


[Nexus6] random movie ad starts too often

I assume this is because of AppLock. I don't mind subscribing to their paid service as long as the price is reasonable, but can't find how to upgrade to it.

7) Account Type
There are three types of account in AppLock. You can choose it in Account Center. It is in the left menu of AppLock.
1. Premium: paid, premium features , no ad.
2. AD: free , premium features , ad.
3. Basic: free, no ad , no premium features.

Even with the above description it was hard to find, but I had to click my Google account string on the "Account Center". That leads me to purchase window.


Unable to resolve your shell environment: Unexpected exit code from spawned shell (code 9, signal null)

Search indicated that this specific issue has not been reported - a related one that does occur because of timeout does exist, is well known, and as such I'm creating a new issue as documentation suggests

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

VS Code Version:

OS Version:

Steps to Reproduce:

Using the following ~/.bashrc, add the last export for resolving some issues with node v18 (i.e. NODE_OPTIONS=--no-experimental-fetching

rik@devrikx:~$ cat ~/.bashrc
# ~/.bashrc: executed by bash(1) for non-login shells.
# see /usr/share/doc/bash/examples/startup-files (in the package bash-doc)
# for examples

# If not running interactively, don't do anything
case $- in
  *i*) ;;
    *) return;;
esac

# don't put duplicate lines or lines starting with space in the history.
# See bash(1) for more options
HISTCONTROL=ignoreboth

# append to the history file, don't overwrite it
shopt -s histappend

# for setting history length see HISTSIZE and HISTFILESIZE in bash(1)
HISTSIZE=1000
HISTFILESIZE=2000

# check the window size after each command and, if necessary,
# update the values of LINES and COLUMNS.
shopt -s checkwinsize

# If set, the pattern "**" used in a pathname expansion context will
# match all files and zero or more directories and subdirectories.
#shopt -s globstar

# make less more friendly for non-text input files, see lesspipe(1)
#[ -x /usr/bin/lesspipe ] && eval "$(SHELL=/bin/sh lesspipe)"

# set variable identifying the chroot you work in (used in the prompt below)
if [ -z "${debian_chroot:-}" ] && [ -r /etc/debian_chroot ]; then
  debian_chroot=$(cat /etc/debian_chroot)
fi

# set a fancy prompt (non-color, unless we know we "want" color)
case "$TERM" in
  xterm-color|*-256color) color_prompt=yes;;
esac

# uncomment for a colored prompt, if the terminal has the capability; turned
# off by default to not distract the user: the focus in a terminal window
# should be on the output of commands, not on the prompt
#force_color_prompt=yes

if [ -n "$force_color_prompt" ]; then
  if [ -x /usr/bin/tput ] && tput setaf 1 >&/dev/null; then
  # We have color support; assume it's compliant with Ecma-48
  # (ISO/IEC-6429). (Lack of such support is extremely rare, and such
  # a case would tend to support setf rather than setaf.)
  color_prompt=yes
  else
  color_prompt=
  fi
fi

if [ "$color_prompt" = yes ]; then
  PS1='${debian_chroot:+($debian_chroot)}\[\033[01;32m\]\u@\h\[\033[00m\]:\[\033[01;34m\]\w\[\033[00m\]\$ '
else
  PS1='${debian_chroot:+($debian_chroot)}\u@\h:\w\$ '
fi
unset color_prompt force_color_prompt

# If this is an xterm set the title to user@host:dir
case "$TERM" in
xterm*|rxvt*)
  PS1="\[\e]0;${debian_chroot:+($debian_chroot)}\u@\h: \w\a\]$PS1"
  ;;
*)
  ;;
esac

# enable color support of ls and also add handy aliases
if [ -x /usr/bin/dircolors ]; then
  test -r ~/.dircolors && eval "$(dircolors -b ~/.dircolors)" || eval "$(dircolors -b)"
  alias ls='ls --color=auto'
  #alias dir='dir --color=auto'
  #alias vdir='vdir --color=auto'

  #alias grep='grep --color=auto'
  #alias fgrep='fgrep --color=auto'
  #alias egrep='egrep --color=auto'
fi

# colored GCC warnings and errors
#export GCC_COLORS='error=01;31:warning=01;35:note=01;36:caret=01;32:locus=01:quote=01'

# some more ls aliases
#alias ll='ls -l'
#alias la='ls -A'
#alias l='ls -CF'

# Alias definitions.
# You may want to put all your additions into a separate file like
# ~/.bash_aliases, instead of adding them here directly.
# See /usr/share/doc/bash-doc/examples in the bash-doc package.

if [ -f ~/.bash_aliases ]; then
  . ~/.bash_aliases
fi

# enable programmable completion features (you don't need to enable
# this, if it's already enabled in /etc/bash.bashrc and /etc/profile
# sources /etc/bash.bashrc).
if ! shopt -oq posix; then
if [ -f /usr/share/bash-completion/bash_completion ]; then
  . /usr/share/bash-completion/bash_completion
elif [ -f /etc/bash_completion ]; then
  . /etc/bash_completion
fi
fi

# Add the GPG key to our profile
export GPG_TTY=$(tty)

# Modify the path variable to contain paths to executables we'd like
# added

# Set the npm global path (prepend it so its found first!)
NPM_PACKAGES="${HOME}/.npm-global"

export PATH=$NPM_PACKAGES/bin:$PATH

# Preserve MANPATH if you already defined it somewhere in your config.
# Otherwise, fall back to `manpath` so we can inherit from `/etc/manpath`.
export MANPATH="${MANPATH-$(manpath)}:$NPM_PACKAGES/share/man"

# Set the android-studio binaries path
export PATH=$PATH:/opt/android-studio-canary/bin
export PATH=$PATH:/opt/android-studio/bin

# Set the android-studio platform tools path
export PATH=$PATH:~/Android/Sdk/platform-tools
#export PATH=$PATH:~/Android/Sdk/platform-tools

# Set the android-studio tools path (don't really need these from terminal)
# export PATH=$PATH:~/Android/Sdk/tools/bin

# Set the GOPATH
export PATH=$PATH:/usr/local/go/bin
export PATH=$PATH:/home/rik/go/bin

# Fix some issues with Node.js v18 while we wait for resolution:
# (Added 6-2022, after there was an error trying to build @kwaeri/filesystem module)
export NODE_OPTIONS=--no-experimental-fetch

Start Visual Studio Code in Gnome DE, leveraging the gui icon (either from panel or favorites):
/usr/share/code/code --unity-launch %F

Useful Information:
The only other thing I can think of, is that at one point in time I did take some steps to try and change the terminal that was opened by opening terminal to a directory from nautilus (file-roller), from gnome-terminal to terminology.
None of the steps did manage to make that happen:

Using update-alternatives:

Using dconf configuration:

gsettings set org.gnome.desktop.default-applications.terminal exec /usr/bin/terminology
gsettings set org.gnome.desktop.default-applications.terminal exec-arg "--current-directory"

Using nautilus-configuration-tool

Using filemanager-actions

However, none worked - so I've put (1) back to gnome-terminal, undone the steps in (2), and uninstalled both (3) and (4).
The issue has started since last update after I updated Node.js from 16 (LTS) to 18, and added the NODE_OPTIONS fix (--no-experimental-fetch).

Looks like exit code 9 is a EBADF (bad file descriptor). Here's what is actually called on startup to fetch the shell environment:
https://github.com/microsoft/vscode/blob/96f310e3ebee05189f03e974713ec1c666b15b6a/src/vs/platform/shell/node/shellEnv.ts#L126-L151
Which runs something close to:
$SHELL -ilc "'<vscode binary path>' --ms-enable-electron-run-as-node -p 'JSON.stringify(process.env)'"

Does removing the --no-experimental-fetch thing fix it? Changes to NODE_OPTIONS will affect the command above.

It does fix it, but that's unfortunate because on Node v18 (at least for now) several modules aren't building when leveraging the latest module versions - with the following error:

Which requires me to export NODE_OPTIONS=--no-experimental-fetch with a follow-up unset NODE_OPTIONS to resolve issues for a successful build.
I tried to change my ~/.bashrc entry to:
# ...
# Set no experimental fetching for resolving node v18 issues:
export NODE_OPTIONS="$NODE_OPTIONS --no-experimental-fetch"

So as not to unset options that vscode might be setting, but that did not resolve the error.  Is that not going to be something that can be avoided?

Though the above also seems like a very odd behavior, considering that according to the official docs:

Options from the command line take precedence over options passed through the NODE_OPTIONS environment variable.

This is just how NODE_OPTIONS works, it overrides options for any node instance that gets run, and that includes VS Code. The approach I'd take here is to set NODE_OPTIONS conditionally depending on what the purpose of the shell is being run is. For example you could do this:
if [ "$TERM_PROGRAM" == "vscode" ]; then
  export NODE_OPTIONS="$NODE_OPTIONS --no-experimental-fetch"
fi

That will set it only when running in VS Code's terminal, this won't work for other terminals of course but it should fix the problem of VS Code's environment being tampered with.


[zeromq] fix warning

[x] Changes comply with the maintainer guide
[x] SHA512s are updated for each updated download
[x] The "supports" clause reflects platforms that may be fixed by this new version
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

Fix build warning "The following variables are not used in CMakeLists.txt: WITH_PERF_TOOL"
The WITH_PERF_TOOL options is set in upstream CMake only for shared builds.

The simpler solution is to use MAYBE_UNUSED_VARIABLES.
It already lists USE_PERF_TOOLS - is this var still used?

USE_PERF_TOOLS replaced it with WITH_PERF_TOOL in MAYBE_UNUSED_VARIABLES.
Also searched now for the whole list of all options in the zeromq CMake and all other options should be still up to date.


Update ownerapi_endpoints, document 2 endpoints, update docs 📚

Changes:

Update ownerapi_endpoints.json to version 4.14.3-1431, on par with the Tesla App.
Add two endpoints to the documentation.

Document /api/1/users/vault_profile.
Document /api/1/users/feature_config.

Split out non-vehicle related endpoints out into a sperate user section of the docs.

These were previously inside docs/vehicle/misc.md.

Remove mentions of retractable/extendable Model X handles. (MX doesn't have door handles, unlike the S)

Notes:

On a seperate note, I have found what appears to be the correct way of requesting the door controls for the Palladium (refreshed/post-plaid) Model X. This was mentioned in #600 Add Support for Model X door controls.
However it is untested, as none of the participants have a Palladium MX to test this.
It seems like it is the correct way, as it was 1. Found in the decompiled Tesla Apk. And 2. It yields a response about missing busses / not being able to wake them, which means that the endpoint was programmed to respond to this specific parameter.

I didn't want to add this information to the docs, as it is pretty much untested, however if the repo owner (@timdorr) wants this to be added regardless, I can do that. (So leave a comment and I will add this to the PR.)
Maybe a contributor or anyone reading this who has a Palladium MX could test it and leave a comment. Instructions below. (M3/Y Testers are also welcome to test my theory.)

The same way we request the Model X door controls, might be the way to also unlatch the M3/Y driver door.

Instructions for the MX Palladium Door Controls (and maybe M3/Y door unlatch)
POST /api/1/vehicles/{id}/command/door_unlock
Opens the door.
POST /api/1/vehicles/{id}/command/door_lock
Closes the door.
To control a specific door you need to add the door parameter along with one of the below values, as JSON into the POST body.
Example
{
  "door": "df"
}

Door
Value

Front door, drivers side
df

Rear door, drivers side
dr

Front door, passenger side
pf

Rear door, passenger side
pr

For the M3/Y door unlatch it might be the same but just that it always has the value df for the parameter.

Thanks!


Having trouble running the Switch version.

There's no README on how to run the Switch version. Could you create one, please? Thanks!

Hi! I'm not familiar with how to run homebrew on the Switch, but basically, you need to put the executable into the root of the SD card, and create a "Data" folder also in the root and put all the PC assets in there. Tell me if that works for you.


Animated Dropdown Menus

Continuation of #56021 to meet guidelines for feature-requests.
In light of new custom controls, I feel that the small fades dropdowns have could have some additions. To detail them individually :

Sub Dropdowns

Sub dropdowns (ones indicated with an arrow), when hovered on, could have the menu shoot out from the side and exalnd down to full height, where the items would then fade in. This is actually similar to some Microsoft Apps that I've used also.

Menu Uncollapsing

Similar to No. 1, when menus are toggled, they could exapnd to max width and then max-height with the items appearing afterwards.

Covering the individual menu items and menu dividers

They should slide in and fade from translarent to opaque. The distance for slide can be small but large enough so that it is opaque when sliding stops, and vice versa.

Possibly? Transition betweem adjacent dropdowns
Similarly to Office 2016, when a tab is clicked, it transitions the ribbon to have the new items. This could possibly be done in VSCode but may be hard since all dropdowns have separate instances, rather than re-rendering the same element
Let's say a dropdown such as File is active, then Edit is clicked: it would fade out the menu items and slide to them left, move the box over and fade in the new items and slide them in from the right. It would go the opposite way if it was from Edit to File.
These are purely suggestions and it doesn't matter if they aren't considered. Thanks for reading

Thanks for the suggestions! The dropdown menus that we use for macOS are from Electron (which have no customization options), but the Windows/Linux custom menu might be able to explore this.


[Android]: makeImageFromView doesn't capture react components that are overlapping with skia components

Description
Hello! We have an issue with making snapshots from views that have both skia images and overlapping react components. On iOS this seems to work perfectly, but on Android only the Skia components are visible in the screenshot and the react elements seem to have disappeared.
Please find below an example repo and video displaying this. In our actual project where we need to implement this, we have a skia image that covered the whole screen with several react components positioned absolutely on top of it, but those components disappear entirely when using makeImageFromView on Android.
Any help to solve this would be highly appreciated!
Thanks in advance,
Jurian
Version
0.1.193
Steps to reproduce

Clone repo:
https://github.com/laurens-lamberts/react-native-skia-example

Run yarn android

Press the lime-colored button in the top right to make a screenshot of the view

A view with a red border appears, which is the snapshot that is generated

Press the lime-colored button again to close the screenshot

You can try to select the different tabs to see the behaviour in different circumstances. For example, in the compass example you can vaguely see the 'take snapshot' button, which in itself is a react component which should have been fully included in the snapshot. On the flappy example. this button disappears completely.

If you'd like to compare it with iOS, please be advised that React-Codegen pod should have a deployment target of at least 12.4 (as a quickfix for this specific test repo)

https://github.com/Shopify/react-native-skia/assets/125256202/91666e26-e4db-4300-b9bd-d75af846dca5
Snack, code example, screenshot, or link to a repository
https://github.com/laurens-lamberts/react-native-skia-example

Hi, @JDBoots - thanks for providing this issue. There is a difference in how Android and iOS handles screenshots, we've tried to follow the implementation from react-native-screenshot - could you try using this library first and see if It has the same issues?

Yeah, this makes sense - on Android you basically have multiple different types of views (regular Views, Texture-based views, OpenGL based - and unfortunately they need to be handled individually.
If you could create a super simple example (I know you have included a link to your other examples) that fits into a single tsx-file without any other dependencies than RN Skia (if possible) that in the smallest way possible reproduces the problem - we'd be able to take a look at why it is not working and maybe even find some kind of solution to the algorithm.

Hi @JDBoots - awesome example - I even added a few additional "hard" parts here:
import {
  StyleSheet,
  Text,
  TouchableOpacity,
  View,
  useWindowDimensions,
} from "react-native";
import React, { useRef, useState } from "react";
import {
  Canvas,
  Image,
  SkImage,
  makeImageFromView,
  useImage,
} from "@shopify/react-native-skia";

const PUPPY_TO_MAKE_YOUR_DAY_BETTER =
  "https://images.squarespace-cdn.com/content/v1/54e7a1a6e4b08db9da801ded/7f2dae36-5650-4b84-b184-684f46fe68aa/98.jpg";

const styles = StyleSheet.create({
  pinkContainer: {
    backgroundColor: "#00ff6922",
    position: "absolute",
    alignSelf: "center",
    left: 50,
    top: 50,
    right: 50,
    bottom: 50,
  },
  salmonContainer: {
    backgroundColor: "#ff8c6922",
    position: "absolute",
    alignSelf: "center",
    padding: 20,
    marginTop: 80,
  },
  limeContainer: {
    backgroundColor: "lime",
    position: "absolute",
    bottom: 20,
    padding: 20,
    alignSelf: "center",
  },
  black: {
    color: "black",
    textAlign: "center",
  },
  innerContainer: {
    backgroundColor: "#ff8c0044",
    transform: [{ translateX: 50 }, { rotate: "45deg" }],
  },
});

const App = () => {
  const { height, width } = useWindowDimensions();
  const image = useImage(PUPPY_TO_MAKE_YOUR_DAY_BETTER);
  const [snapShot, setSnapShot] = useState<SkImage | null>(null);
  const snapRef = useRef<View | null>(null);

  const makeSnapshot = async () => {
    const newSnap = await makeImageFromView(snapRef);
    setSnapShot(newSnap);
  };

  const resetSnapshot = () => setSnapShot(null);

  return !snapShot ? (
    <View ref={snapRef}>
      <View style={styles.pinkContainer} />
      <Canvas style={{ width, height }}>
        <Image
          image={image}
          height={height}
          width={width}
          fit={"cover"}
          opacity={0.5}
        />
      </Canvas>
      <View style={styles.salmonContainer}>
        <Text style={styles.black}>Let me be a part of your snapshot!</Text>
        <View style={styles.innerContainer}>
          <Text style={styles.black}>I'm inside the red thingie!</Text>
        </View>
      </View>
      <TouchableOpacity style={styles.limeContainer} onPress={makeSnapshot}>
        <Text style={styles.black}>Make snapshot</Text>
      </TouchableOpacity>
    </View>
  ) : (
    <View>
      <Canvas
        style={{
          width: width,
          height: height,
        }}
      >
        <Image
          image={snapShot}
          x={50}
          height={height - 100}
          width={width - 100}
          y={50}
        />
      </Canvas>
      <TouchableOpacity style={styles.limeContainer} onPress={resetSnapshot}>
        <Text style={styles.black}>Reset snapshot</Text>
      </TouchableOpacity>
    </View>
  );
};

export default App;

Have found some ways to improve - will try to create a PR!

I did try the example on Android before merging but I must have not paid attention because the example doesn't seem to work at all on Android. My apologies for that.

@chrfalch I am seeing the same thing than the screenshot indeed.


Usage example not working

I'm using the example in the 'Usage' section. Installed springroll with bower.
I downloaded 'preloadjs.min.js' and included it as another dependency.
When I open my web page I keep getting:
Uncaught Unable to include 'createjs.Bitmap' in core.min.js:2
What am I doing wrong? Is there a complete and working example I can clone?

ok whatever, I downloaded the examples and will use that to kickstart the project. But it looks nothing like the basic usage example.
I think the following link was needed in my first attempt, but this wasn't mentioned as a dependency in the instructions:

This is a SpringRoll 1 issue, closing


Incorrect version showing for v1.19.65

ℹ Computer information

VFP version: 9
FoxBin2PRG version: v1.19.65
Last FoxBin2PRG version w/o bug (if available): v1.19.63

📝 Provide detailed reproduction steps (if any)
When running Txt -> Bin the progress bar is showing v1.19.64 even though v1.19.65 is installed.
Also this repo has releases till only v1.19.62 and unable to find later releases and notes.
✔ Expected result

Should show v1.19.65 on title bar when running FoxBin2Prg
Is it possible to have releases and notes in Github?

❌ Actual result

Showing v1.19.64 on title bar when running FoxBin2Prg v1.19.65
Releases notes unavailable in Github for >v1.19.62

📷 Screenshots

Sure, we could push the changes.

@siara-cc please do issues in one problem one issue form. It's hard to address otherwise.

Also this repo has releases till only v1.19.62 and unable to find later releases and notes.
This is only the tags are missing. The code is recent. Since I provide the code and Doug is pushing, there are flaws from time to time. Sorry about this.

@siara-cc

Is it possible to have releases and notes in Github?

I do not grok this.  What do you expect?
This repo is basically for deploying FoxBin2Prg via Thor, and to provide the source for participating. We do not create releases in the sense of deployment other then deploying via Thor. I do not know what github is doing under Releases - there are simply the tags listed.
I was also a bit surprised to found the question under Expected result section. I tend to ignore this if the problem is clear from the issue name. :)

Hi @lscheffler ,
Thank you for addressing the issue.  About the release notes, I thought it could be useful if the changes are mentioned in the description of Releases section, for people like me trying to figure out if we need to upgrade to a particular version.
The version displayed in the progress dialog is useful to show that I am using the correct version when discussing with other developers, particularly differences in files due to recent changes in format of DB2 files.  So I used the prescribed template to report the issue here.
I did not realise the problem faced because of reporting two issues in one. I will keep this in mind when reporting issues in future.
I would like to thank you for the time spent in looking into this and also for providing us a great tool !!
@DougHennig

@siara-cc
You are perfectly fine with the problems. The version number and the change list issues are solved.
The tag issue not (This is the links from the change-list will not work, for example) Normally the change-list shows the recent state of this repo. The changes are also visible via Thors update process.
I recommend  to use the latest version anyway, it's not a big deal to download via Thor or git. I try to address all changes, but sometimes minor hiccups will be changed quietly.


Web UI becomes unresponsive

Despite the fix in #2, it'd appear the web UI can still become unresponsive after a period of time. No further information is available except 'refused to connect' error is received by the client.

#43 contains what might be a trace, or might not...

http://stackoverflow.com/questions/180095/how-to-handle-a-broken-pipe-sigpipe-in-python/
http://stackoverflow.com/questions/12591760/flask-broken-pipe-with-requests

Added threading in 6da4f4f6fe40923f0398b48c0c6d6fd268a89e83 let's see how this goes...

http://stackoverflow.com/questions/37962925/flask-app-get-ioerror-errno-32-broken-pipe

I've rather happy with 6da4f4f, however it must be noted that this needs to be removed in production and a proper front end web server used. As such, this bug is fixed and moved to enhancement (Documentation)

Ungh not closed... just reassigned....


Add comments for decoding memo and search attr

Pull Request Test Coverage Report for Build 3d7c7700-8f0d-4106-b503-58b05b2637c2

0 of 0   changed or added relevant lines in 0 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 75.751%

Files with Coverage Reduction
New Missed Lines
%

internal/common/convert.go
3
84.62%

Totals

Change from base Build 9bc04ebd-d422-40c6-bfdf-4d0ece286fa6:
-0.03%

Covered Lines:
8925

Relevant Lines:
11782

💛  - Coveralls


Support XML responses for WCS GetCoverage

The WCS 1.1.1 spec supports returning a GetCoverage response as an XML document according to https://portal.opengeospatial.org/files/07-067r2
We currently implement:
10.3.1 Introduction
The normal response to a valid GetCoverage operation request shall be a single coverage extracted from the coverage requested, with the specified spatial reference system, bounding box, size, format, and range subset.

But we could also support:
10.3.11 Response encodings
<IP_ADDRESS> Overview
The encoding of the GetCoverage response consists of a Coverages XML document (see <IP_ADDRESS> below), packaged and bundled according to the request encoding and the value of the ―store‖ parameter (see Table 24). WCS servers shall implement response encodings according to Table 31.

This additional feature would allow us to support much more complete TimeSequence queries for arbitrary lists of TimePeriods or TimeIntervals.
It's likely that this implementation would also require implementing the store parameter to fully support the spec.
To verify that these types of requests are not currently supported by gt server, I made a request to:
http://localhost:9000/?SERVICE=WCS&VERSION=1.1.1&REQUEST=GetCoverage&FORMAT=xml&IDENTIFIER=us-ned&<...>
And got:
Error parsing parameters: Unsupported format: 'xml'.

Noting here that it doesn't look like GeoServer supports temporal queries at all, their WCS Request Builder demo doesn't allow you to choose a time slice:
https://docs.geoserver.org/stable/en/user/services/wcs/requestbuilder.html#wcs-request-builder
QGIS supports the TimeSequence parameter, at least it exists in the code. Further testing in #157 might reveal further details.


[DO NOT MERGE] Reference refactoring of BaseInnerProductAttributor

Description
This PR is a reference implementation to refactor the BaseInnerProductAttributor, which is NOT intended to be merged (and has not been tested). As an example, the Arnoldi attributor is re-implemented to demonstrate how the new API of BaseInnerProductAttributor can be used.
A few major changes in BaseInnerProductAttributor:

transformation could now be applied to both train data representations/gradients and test data representations/gradients
the batching over the full training data for transformation was original dealt inside .attribute() with a simple average, which lacks enough flexibility; it is now moved into the transform functions
.cache() could do more than just storing the full training dataloader. For example, the Arnoldi projector is now processed and cached in transformation_on_query in a hacky way. In this PR, this is moved to .cache()
removed transformation_kwargs and added an init for the subclass. It's better to specify these attributor hyperparameters explicitly than relying on a kwargs

@TheaperDeng, one thing I'm not sure is if we want to set the default transform functions for both train and test as identical transformation? As in many cases, only test rep needs to be transformed.

@TheaperDeng, one thing I'm not sure is if we want to set the default transform functions for both train and test as identical transformation? As in many cases, only test rep needs to be transformed.

It happens that I am working on this. I think one of the possible way is that, we just make transform_test_rep to be abstractmethod, that is users always need to implement this one. and for transform_train_rep, we can have a default implementation that returns the train rep directly.

@TheaperDeng, one thing I'm not sure is if we want to set the default transform functions for both train and test as identical transformation? As in many cases, only test rep needs to be transformed.

It happens that I am working on this. I think one of the possible way is that, we just make transform_test_rep to be abstractmethod, that is users always need to implement this one. and for transform_train_rep, we can have a default implementation that returns the train rep directly.
Or, we directly set them both to be defaultly return the rep directly. Users can override them when design their TDA methods.

I think the latter is Ok. Users implementing new attributors should be a bit more tolerant than users who just use the attributors, and they don't need an abstract method to remind them this is something they should override.

@TheaperDeng, one thing I'm not sure is if we want to set the default transform functions for both train and test as identical transformation? As in many cases, only test rep needs to be transformed.

It happens that I am working on this. I think one of the possible way is that, we just make transform_test_rep to be abstractmethod, that is users always need to implement this one. and for transform_train_rep, we can have a default implementation that returns the train rep directly.
Or, we directly set them both to be defaultly return the rep directly. Users can override them when design their TDA methods.

I think the latter is Ok. Users implementing new attributors should be a bit more tolerant than users who just use the attributors, and they don't need an abstract method to remind them this is something they should override.

Yeah, I think so. And this makes the innerproduct IF can be directly instantiated and works like Grad-Dot, just like its name.

@tingwl0122 could you please take a look at the new implementation of the Arnoldi attributor and see if there is anything wrong?

@tingwl0122 could you please take a look at the new implementation of the Arnoldi attributor and see if there is anything wrong

I think the overall structure looks good to me. (Moving projection matrix computation to cache has been listed as one todo) Another thing is that the original Arnoldi paper actually used only a random batch of the full dataloader to do the algorithm, which I didn't include before. We can do something on the data_target_pair before feeding it into func to achieve this now. (also adding an additional param to control this random batch's size)

@tingwl0122 could you please take a look at the new implementation of the Arnoldi attributor and see if there is anything wrong

I think the overall structure looks good to me. (Moving projection matrix computation to cache has been listed as one todo) Another thing is that the original Arnoldi paper actually used only a random batch of the full dataloader to do the algorithm, which I didn't include before. We can do something on the data_target_pair before feeding it into func to achieve this now. (also adding an additional param to control this random batch's size)

another minor thing is that we will need to force users to use .cache before attributing. We can probably raise some errors for this.

@tingwl0122 could you please take a look at the new implementation of the Arnoldi attributor and see if there is anything wrong

I think the overall structure looks good to me. (Moving projection matrix computation to cache has been listed as one todo) Another thing is that the original Arnoldi paper actually used only a random batch of the full dataloader to do the algorithm, which I didn't include before. We can do something on the data_target_pair before feeding it into func to achieve this now. (also adding an additional param to control this random batch's size)

another minor thing is that we will need to force users to use .cache before attributing. We can probably raise some errors for this.

Sounds great. Thanks for the notes. I'll add them later.

@tingwl0122 could you please take a look at the new implementation of the Arnoldi attributor and see if there is anything wrong

I think the overall structure looks good to me. (Moving projection matrix computation to cache has been listed as one todo) Another thing is that the original Arnoldi paper actually used only a random batch of the full dataloader to do the algorithm, which I didn't include before. We can do something on the data_target_pair before feeding it into func to achieve this now. (also adding an additional param to control this random batch's size)

another minor thing is that we will need to force users to use .cache before attributing. We can probably raise some errors for this.

Sounds great. Thanks for the notes. I'll add them later.

Thanks @tingwl0122 @jiaqima I think I can directly make the changes in my implementation #133

Closing this PR in favor of #133. Any follow-up discussion could be done there.


Setup glances on nixos

To enable monitoring of disk/ram/cpu in homeassistant. It can easily be installed but need to figure out how to start it? Systemd?

Done!


Let Annotations be schema:Things

As discussed with @alastair, model TROMPA Web Annotations as Things (as well as Annotations), probably using additionalType, so that we can store them within ItemLists

@musicog can you explain a bit more detail how additionalType may work?
At the moment when we render an item to json-ld, its type is that which is defined a the top of the schema file.
For example, a DigitalDocument is defined as being both a Thing and a CreativeWork:
https://github.com/trompamusic/ce-api/blob/staging/src/schema/type/DigitalDocument.graphql
We've not been mandating that these items also have the additionalType field filled (as a single item, as a list?). In this case, is it mandatory that we add both schema.org/Thing and schema.org/CreativeWork to the additionalType field when rendering to json-ld? Where should this be done? Should it be done only in the json-ld renderer? Should we require that this field is always set in mutations? Can we automatically fill it in somehow using the graphql-neo4j bindings that we use?

Your example specifically does not require additionalTypes to be specified since schema:DigitalDocuments are rdfs:subClassOf schema:CreativeWork (which are in turn rdfs:subClassOf schema:Thing) -- see https://schema.org/DigitalDocument.ttl and https://schema.org/CreativeWork.ttl
However, oa:Annotations are not rdfs:subClassOf schema:Thing, since schema.org and the Web Annotations Data Model are not (explicitly, formally) interlinked. Hence why we need to take the trouble to specify the additionalType.
In other words, a web client application that cares about semantics ought to expect schema:DigitalDocuments to be specialisations of schema:Things, but it's unfair to expect them to know that oa:Annotations are schema:Things without the additional prompt.
As to how to render this in json-ld, my first instinct would be to add both the append the type and any additionalTypes of any given node into the "@type" field of the json-ld output - always producing a list, even if it's a list of one item. Unless you foresee a case where we have to retain the separation of "type" and "additionalType" in the JSON-LD, in which case we could directly incorporate the schema:additionalType property into the JSON-LD output. I suspect the former would produce more useful data.

Thanks for the explanation. This distinction is clear to me now.
It'd be nice to make this happen automatically when we create an Annotation using the APi.
@ChristiaanScheermeijer, do you know if it's possible to add some additional logic only to CreateAnnotation so that we automatically fill in a field if it's not provided by the user?

related: #85

I made annotations schema:Things in #100