Arrays within Arrays on multiple HTML elements, only the last one is used

Hi,
I have an array data coming back from my API, within my data is another Array which contains members which I want to display in separate td within a table.
The data looks like the following:
"data": [
	{
		"CallType": "Inbound",
		"CallFrom": "01234567",
		"CallState": "Answered",
		"Date": "21\/07\/2017",
		"Time": "09:10:40",
		"CustomerDetails": {
			"Customer": "Customer Name",
			"Name": "Contact Name",
			"Email"<EMAIL_ADDRESS>		}
	}
]

I am using the template as follows
$('[template="call_log_template"]').template(object.data);
My HTML looks like:

                    Date Called
                    Time Called
                    Call From
                    Customer
                    Contact
                    Email
                    Call State

The issue is that when the table is shown in the 4th column where it should show the customers name, it seems to get skipped over, as does the name, and then only shows the email, followed by the call state column. The customer and the name aren't displayed, so I'm only outputting 5 columns instead of 7.
I've gone through the tutorial but I can't see a way around this at the moment.
Thanks
Chris

Hello Chris,
here is the CodePen Example.
Note that template="[CustomerDetails]" has the meaning of for each item inside CustomerDetails - so in your case it should be placed on tr element rather then on td.

I am closing this issue as this is not a bug. If I missed something reopen it.

Hi,
I'm not sure if I'm missing something, but from your example, this now means that the only things that do get set are the variables within the CustomerDetails array so its now only filled in 3 columns and the others are blank.
What I need to do is columns 1, 2 and 3 are set using the members of the root data array, then 4, 5 and 6 are then set using the members of the CustomerDetails array, then column 7 is set using the member again from the root of the data array.
Unless I'm missing something it doesn't seem like what I need is supported, as arrays within the array can be only used as part of another sub template, which I can then provide children elements to set the variables, which in my case I can't do as the template is set for the data array, I then can't get the values out of CustomerDetails to then set each TD when needed.
Thanks
Chris

Ah, I see!
No, that is not supported because the pre-processing of data are in many cases trivial in Javascript so I didn't consider support for deep referencing of variables (I may if it will be the common use case).
For now it is indeed not supported but it takes just one-liner to achieve that effect:
response.data = response.data.map((v) => $.extend(v, v.CustomerDetails));

I have updated the Codepen example. I also assumed that response.data array will contain more items so I have fixed it so it works with multiple lines.
Check it out and let me know if that's it.

Ah yes, that's working perfectly.
Thanks for your help


Elasticsearch compatibility

One use case for mustache is building parameterized search templates for Elasticsearch. The rendering engine in Elasticsearch has two special built-in functions/annotations:
{{#toJson}}some-object{{/toJson}} renders the nested object to valid JSON, and {{#join}}some-array{{/join}} concatenates the strings in the array to a single string with a default or custom delimiter. Could chevron implement those annotations as well? Asking for a friend :)

Would writing a toJson and join lambda in "user space" work for your use case? That should give you full control over your desired functionality without changing behavior for non-elastic search users.
https://github.com/noahmorrison/chevron/blob/90413a2dd56e06e1a493aa703907febe43966ca3/test_spec.py#L289 for reference.
Any thoughts or additional requirements? Thanks!

Would writing a toJson and join lambda in "user space" work for your use case? That should give you full control over your desired functionality without changing behavior for non-elastic search users.
https://github.com/noahmorrison/chevron/blob/90413a2dd56e06e1a493aa703907febe43966ca3/test_spec.py#L289 for reference.
Any thoughts or additional requirements? Thanks!

That should work for the use case, if I can offer any help feel free to get in touch

That should work for the use case, if I can offer any help feel free to get in touch


Sort security_groups.tf output

Coverage remained the same at 99.786% when pulling 51c93e9d701a0c8f99e897ac1a4836adb690eae8 on dancadar:sort_security_groups.tf into f5e482afce717d70f5edc5735ce9ace32d6e99e8 on dtan4:master.

I cannot accept this pull request now, because we have to do the same for all other resources.
If you have a strong reason to sort results, please explain it.
Thank you!

Tracking changes in the environment would be much easier if the output is always sorted in the same way - a simple diff between an older tf file and a fresh tf file. For now I'm focused only on security groups, so I ignored the other resources, that's why the PR was only for security groups template.


Parsers should record original character encoding on ASTs

Currently we always print ASTs out UTF-8 encoded. This can be a problem when writing the results of recipe execution back to a non-UTF-8 file. If parsers recorded the original encoding on the AST, then at printing time we could write back to the file in the same encoding it came in.

Fixed by: https://github.com/openrewrite/rewrite/commit/255967d1624da16bdf4606ae9af6ecb42051fd15


CLNN master/thnn4 build error

Hi Hugh,
I believe there is a link error in the current thnn4 / master branch.
The function GET_BLOCKS is in common.h, which has been compiled correctly

[ 25%] Building CXX object CMakeFiles/clnn.dir/common.cpp.o
[100%] Linking CXX shared module libTHCLNN.so
Undefined symbols for architecture x86_64:
"GET_BLOCKS(THClState*, int)", referenced from:
_THNN_ClSpatialAveragePooling_updateOutput in SpatialAveragePooling.cpp.o

Thanks in advance for the help.

It builds ok for me.  Can you do the following and check the problem does/doesnt still occur:
luarocks install torch7
luarocks install nn
luarocks install cltorch
luarocks install clnn

Ah, seems travis has the same issue, https://travis-ci.org/hughperkins/clnn/builds/107116666  Hmmm.

Can you confirm what platform you are on please?

I am on Mac OS 10.11.3
Something I may be missing:

luarocks install torch7

Does not work for me (No results matching query were found.), neither with the luarocks embedded in my already installed torch, nor with the "native" luarocks from Lua 5.2.
To confirm, all the master branches are now in sync - so we can disregard the "bleeding edge" instructions?
Lastly, I ran

luarocks install torch
luarocks install nn
luarocks install cltorch
luarocks install clnn

Where clnn did not build.

sorry, I meant luarocks install torch, corrected this above.

Corrected as well.

I am on Mac OS 10.11.3

yes, I think it's a problem specific to Mac OS X.  Luckily, I do have Mac OS X build on travis, so I can try .. stuff... see if I can fix it.  If you happen to have any ideas in the meantime please let me know.  It problably involves a combination of one or both of the following mac-specific things:

symbols in one dll arent available to later dlls by default, without some work

and non-flat namespace doesnt help here

undefined symbols in a dll cause an error at linktime, whereas in linux, I think they are silently ignored, unless you try calling them

Seems the symbols really are needed, so it's probably to do with the first issue.

I've moved common.cpp/.h into lib/THCLNN directory/dylib.  That hopefully eliminates the build errors with GET_BLOCKS and GET_CL_NUM_THREADS.  Then, (if those are fixed), "just" need to fix the issues with the TemplatedKernel symbols (which are in libEasylCL, I think)

Indeed, the missing symbols are now from the classes TemplatedKernel and EasyCL.

So, basically, on linux, by doing objdump somelib.so | grep NEEDED, starting with build/lib/THCLNN/lib/libTHCLNN.so, we have the following chain of dependencies:

libTHCLNN.so => depends directly on libTHCl.so
libTHCl.so => depends directly on libEasyCL.so

Unfortunately, libTHCLNN.so needs a couple of symbols from libEasyCL.so, and, on Mac, these are not available to libTHCLNN.so by default.  Needs some magic linking options I think.

Sorry I am not managing to understand what cmake command the rockspec file is going to trigger.
If it's C/C++ compilation only I should be able to debug.

Ah, the easiest way to try it I think is something like:
git clone https://github.com/hughperkins/clnn.git
cd clnn
luarocks make rocks/clnn-scm-1.rockspec

Once you've done this once, then you can run cmake directly:
cd build
make

Thanks, as far as I know the directory where shared libraries are looked up is different on Mac.
But it could just be that the install/lib/ torch folder does not have a libEasyCL.so, but a libEasyCL.dylib (http://stackoverflow.com/questions/2339679/what-are-the-differences-between-so-and-dylib-on-osx)

Yeah, when I say .so on mac, that will sometimes be .dylib, sometimes .so, depending on the linker options supplied.

I've added EasyCL as a direct dependency of THCLNN, maybe that will help.  But I should go to bed now-ish really, so I will check in the morning.  https://github.com/hughperkins/clnn/commit/19ab3b5f8d7229cefe73e88f92782f97827ecb05

Built ok :-)  https://travis-ci.org/hughperkins/clnn/builds/107126113

Built ok too, thanks for the help.

Cool :-)


ImportError: attempted relative import beyond top-level package - Windows 10

Describe the bug
(venv) PS C:\Users\Farma\chiadog> .\start.cmd
Traceback (most recent call last):
  File "C:\Users\Farma\chiadog\main.py", line 12, in <module>
    from src.chia_log.handlers.daily_stats.stats_manager import StatsManager
  File "C:\Users\Farma\chiadog\src\__init__.py", line 5, in <module>
    from ...parsers.finished_signage_point_parser import FinishedSignagePointMessage
ImportError: attempted relative import beyond top-level package
(venv) PS C:\Users\Farma\chiadog>

Tried without venv too. All requirements installed. Config checked few times.
Tried starting with
python.exe main.py --config 'C:\Users\Farma\chiadog\config.yaml'
python.exe main.py --config config.yaml
Same error.
Environment:

OS: Windows 10
Python version: 3.9
PIP version: 21.1.3
Chia version: 1.1.7

Try a full reinstall of git repo?
It's trying to go up three folders from src/__init__.py which is not possible.
It seems your install got corrupted somehow.

Agree with @pieterhelsen's suggestion. Closing for inactivity.


Items from player's inventory are thrown forwards on death, rather than scattered in all directions

This seems related to Issue #28, as it's another case of items spawning with the wrong velocity. I haven't tested whether or not sneaking while dying will launch the items with extra velocity.

This is now fixed in dropz 2.1.1.


Improve Timer ergonomics

I often use timers to drive animation by converting it to a percentage. The fact that elapsed could exceed duration for non-repeating timers means that I had to check for that everywhere I used them, or the percentage could go negative (which can do various unexpected things, such as flip vector direction). This is a paper cut for me, so I made this PR which:

Clamps elapsed to the range [0.0, duration]
Adds tests
Adds some documentation
Adds two convenience methods: (percent and percent_left)

I was on the fence whether to include these methods in the PR due to the fact that it assumes keeping the size of Timer small is more important than the speed of the calculation. A more speed-optimized approach would store the inverse of duration and do multiplication.  Is there a general principle we're following that could inform the approach? Or should we leave the methods out altogether?

Here's an example from a showcase I'm making that demonstrates using these changes to directly power a shrinking death animation and a punching boxing glove animation.

Ooh I love the look of that boxing game.

I was on the fence whether to include these methods in the PR due to the fact that it assumes keeping the size of Timer small is more important than the speed of the calculation. A more speed-optimized approach would store the inverse of duration and do multiplication. Is there a general principle we're following that could inform the approach? Or should we leave the methods out altogether?

In this case I prefer the clarity of division over storing / multiplying by the inverse.
Using percentages seems like a common enough requirement for timers that I think its worth including, largely because the function name helps encode intent.
I think this is good to go!

Semi-relevant idea that I'd love to discuss with yall (I'm pulling in @amberkowalski because they're also working on timers):
I think it might be a good idea to remove the "auto ticking" timer system.

Resource-based timers don't auto-tick
Component-based timers only auto-tick if they aren't wrapped. But I think having multiple timers with different "names" will be common (see @CleanCut's boxing example).

Its a bit inconsistent that only raw Timer components auto-tick and theres no good way to force all timers to tick. I think I'd rather just make ticking explicit everywhere for simplicity.

@amberkowalski if we make ticking explicit everywhere, maybe we don't need built in pausing? users could "pause" by not ticking where relevant according to their own context-specific logic.

My two cents:

Making ticking of timers always explicit seems like the sanest option. I didn't even think to look for auto-ticking, and would have been surprised had I run into it.  Including an example of how to tick your timer in the docs for Timer should be sufficient.
I would welcome pause functionality -- it is a useful and typical feature to have built-in to the timers. In its absence, a user who needs to pause is faced with either tightly coupling the pause logic to the ticking logic, or wrapping/reimplementing the Timer with pause logic.

In this case I prefer the clarity of division over storing / multiplying by the inverse.
Using percentages seems like a common enough requirement for timers that I think its worth including, largely because the function name helps encode intent.

🥳 🎉

I would welcome pause functionality -- it is a useful and typical feature to have built-in to the timers. In its absence, a user who needs to pause is faced with either tightly coupling the pause logic to the ticking logic, or wrapping/reimplementing the Timer with pause logic.

Good enough for me. @amberkowalski, if you update your pr to account for the changes in this pr I think it'll be merge-ready!

* Making ticking of timers always explicit seems like the sanest option. I didn't even think to look for auto-ticking, and would have been surprised had I run into it.  Including an example of how to tick your timer in the docs for `Timer` should be sufficient.

Having said that, having a way to opt-in to auto-ticking could be very convenient. 🤔 But I think I'm just feature creeping now. 😆

Having said that, having a way to opt-in to auto-ticking could be very convenient.  But I think I'm just feature creeping now.

Would providing a system that auto-ticks all timers and an example how to include it into the App, count as opt-in?

Haha I'm very curious how you would do this across wrapper types without adding some sort of internal shared state

Nyah. Just for the not wrapped ones - like it works now.


Integration issue in dev mode - provider key error

crowd.dev edition
Community (self hosted)
Version
v0.33.0
What happened?
On Integrations page
Cant connect any of the available platforms
https://github.com/CrowdDotDev/crowd.dev/assets/67850763/8a65c63d-4060-434f-a96e-d5fc33c8de96
Error in console

Command / Request
No response
Relevant log output
No response

Hi @Dhruwang, thanks for raising this! We will check this out ASAP and get back to you.

Hey @Dhruwang! Thanks for raising this
To connect integrations in development mode, you must first create and configure an app. You can find the docs here. The reason is that we cannot expose our app secrets to the community.
However, some integrations can be connected without having to do any configuration, such as HackerNews.

Ok got it , Thank you !


allow for differentiating between forge and fabric versions

like the title says unless I have overlooked something I don't see a way to say I only want forge versions for example.
e.g: https://www.curseforge.com/minecraft/mc-mods/xaeros-world-map

Oh, I completely missed this! Seems easy enough.

This should work, but some mods may not have Fabric in their version tags, which could get tricky.


[bug] Installer is not signed

Unfortunately, the installer is not digitally signed. This leads to Windows 11, for example, complaining that this installation package is not trustworthy, which ultimately leads to nothing being installed. You can install it if you explicitly ignore the risk. However, depending on the security guidelines set, this may not be possible.

Still is still in progress w/ the Microsoft/Azure team

https://github.com/onlook-dev/onlook/pull/960


consul_agent RUN_DIR should be created in pre-start

The consul_agent agent_ctl script attempts to create the RUN_DIR directory for the consul_agent.  However since agent_ctl is running as user vcap, it cannot create the directory.
Perhaps the RUN_DIR directory should be created in pre-start and subsequently have its ownership changed to vcap:vcap
join_ctl also has code to mkdir and chown for the RUN_DIR directory

Hi @rboykin
Can you share some more details about the errors you're seeing?  What version of the release are you deploying, and what stemcell?  If possible, a copy of your manifest with sensitive information redacted, and a paste of your bosh deploy output would be helpful.  I believe the RUN_DIR lives on ramdisk and is safe to create in the regular _ctl script.  I'll have to check with the team, but there was some work we did a while back to minimize what we run as root, and there was a reason at the time we couldn't move RUN_DIR also into the pre-start.
Thanks,
Amit

consul_agent would not come up even after pre-start was run until I manually did a mkdir -p /var/vcap/sys/run/consul_agent and then chown vcap:vcap.
Check the agent_ctl script.  It is run as vcap:vcap and it assumes it can mkdir /var/vcap/sys/run/consul_agent.  Bad assumption.
Trying this on a 242 clean deployment.
After doing a monit start consul_agent,
agent_ctl.err.log contents:
[2016-09-15 05:40:56-0500] + main start
[2016-09-15 05:40:56-0500] + mkdir -p /var/vcap/sys/run/consul_agent
[2016-09-15 05:40:56-0500] mkdir: cannot create directory '/var/vcap/sys/run/consul_agent': Permission denied

Creating the RUN_DIR as is was done intentionally, not based on assumptions.  I'll have to get back to you with details. What stemcell are you on?

I am running cf_nise_installer.  It installs directly on an existing VM.  I also note that other monit jobs create their RUN_DIR's within their pre-start scripts, namely all others (at least in cf_nise_installer) that are started from their monit job as vcap:vcap other than consul_agent.
Sorry, did not mean to insult with the assumption comment.

  You'd have
to ask the BOSH community for more details on that, either on
github.com/cloudfoundry/bosh/issues, cf-bosh mailing list (
https://lists.cloudfoundry.org/archives/list/cf-bosh@lists.cloudfoundry.org/),
or #bosh slack (https://cloudfoundry.slack.com/archives/bosh).
We've done it this way due to some combination of factors, namely (a)
pre-start doesn't always run before monit start and (b) we wanted to run
our _ctl script as vcap instead of root.  We had originally put the run_dir
creation in the pre-start script, I'll ask around and try to figure out the
exact issue we hit with that.  I recall we "got lucky" when we found out
BOSH handled tmpfs differently and was safe to create run-dir in the _ctl
script.
Anyone @cloudfoundry-incubator/cf-infrastructure recall the exact details
of this?
/cc @cppforlife
On Thu, Sep 15, 2016 at 6:53 AM, J Russell Boykin<EMAIL_ADDRESS>wrote:

I am running cf_nise_installer. It installs directly on an existing VM. I
also note that other monit jobs create their RUN_DIR's within their
pre-start scripts, namely all others (at least in cf_nise_installer) that
are started from their monit job as vcap:vcap other than consul_agent.
Sorry, did not mean to insult with the assumption comment.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/cloudfoundry-incubator/consul-release/issues/44#issuecomment-247334297,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC9t3pQg0-_6wJkTal59n40q67YGvVw4ks5qqU3egaJpZM4J9Akb
.

Note that in both the postgres and etcd cases specified as examples above, the RUN_DIR is also created in their respective pre-start scripts:
https://github.com/cloudfoundry/postgres-release/blob/master/jobs/postgres/templates/pre-start.erb#L35
https://github.com/cloudfoundry-incubator/etcd-release/blob/master/jobs/etcd/templates/pre-start.erb#L21

Good point.  Will take a look.

Thanks much @Amit-PivotalLabs

Hi @rboykin,
Took a look and it seems like we should do the mkdir RUN_DIR in both the ctl and pre-start so we have added that: https://github.com/cloudfoundry-incubator/consul-release/commit/351bae9ad0ed38b396b9d328bdbb75e81da481f8.
Thanks.

Thanks @christianang


Feature fielddb2.50 integration

Step 1:

loading FieldDB via requirejs
changing the brand depending on the server the user selected
using the Fielddb auth login methods
implementing the confirm, bug and prompt UI hooks

@jrwdunham  please review and test and merge this one first so that i can get you feedback before moving on to the corpus.coffee

@jrwdunham  i removed more cruft, its resuming auth sessions on my computer and other events are working (ie, you login once, and refresh and it knows who you are) it doesnt show your list of corpora yet because i need want to get the auth approved before i move on to the corpora select process.

@jrwdunham please click the restart button on the build. there was a dependacny missing in fielddb when the last commit was sumbitted, its there now.

@jrwdunham if you're still awake can you give some feedback on this pull request... all other apps have been updated to use the commonjs library now, if you don't start using it soon you will be receiving data you have to do extra work to handle. it also reduces the number of server requests.

@jrwdunham  i did the changes and added some notes for you. I also implemented the bug, confirm and prompt dialogs so you have some extra code to look at in terms of coffeescriptification.

please take a look at this if you have time today :) i'm not jet lagged yet so i can work on this stuff if you want

need to make the connection return the authUrl as

url so you can edit it in the app settings

@jrwdunham even setting server1 to an object in Dative i cant get the url to be set by the time it gets to the serverscollection.

( i gave up on using the fielddb connections because i found that  the Server class wont accept any extra attributes anyway)

@cesine logging out from FieldDB is not working. Under logoutFieldDB at line 272 of models/application-settings.coffee, @get('fieldDBApplication').authentication.logout is undefined.

can you paste the version of fielddb here? maybe you didnt run the grunt update tasks that i made for you. just incase you can also paste the contents of @get('fieldDBApplication') here to see if its actually defined too

@jrwdunham  i think i might have found the problem. If we call fetch() in the initialize, then the @get('fieldDBApplication') wont be empty:

Does this seem likely and/or feasible as a solution? or is there a fetch() somewhere prior/after to the initialize that should have been called?


Printing from IPAD in A4 prints in 1/4 sheet

Hi, im trying to print with ur library, but i have a weird bug.
In the preview, "my design" occupies full sheet, but when i print, it is printed in 1/4 sheet

Any tip?

I just check that it happends with border:false in options

Usually border:false is the better choice for images but good that you found a way.


I would like to join the organization

Please send me an invitation as a member of Eddie Jaounde community

Invitation sent for the GitHub Organisation. Welcome to the community 🎉
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see, you can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/EddieJaoudeCommunity/people
Tips for practising:
Customise your GitHub profile, here is a live stream on it https://www.youtube.com/watch?v=cT6GXCuS0Zo
Practice repo, instructions on how to add your name to the README in the README https://github.com/EddieJaoudeCommunity/hacktoberfest-practice
Remember contributing to open source is not just about code, its about collaboration, communication and adding value
I hope that helps


SSR causes duplicate graphql requests

I'm really interested in using Relay for a new project I'm building, and after testing out isomorphic-react-router it seems the spiritual successor is found-relay with SSR. I tried out the found-modern-universal branch locally, and my biggest concern is that the number of graphql requests to the server would double on initial page load. Besides being bad for the server, that's not great for the client either, and should technically be unnecessary since the client already has all the data it should need inside window.__RELAY_RECORDS__ from what I can see.
Given this, there are a few possibilities for what's happening:

This is a known issue, and perhaps already on your roadmap to add/fix?
This is a bug that I'm experiencing that isn't known -- seems unlikely since I've only added some debugging code so far, but maybe I have an unsupported version of node/npm/whatever.
This is totally unavoidable with Relay Modern -- I really hope this isn't the case.

I'm curious which one of those it is, and if there is an existing or forthcoming solution.
Thanks!

Oh, it's a bug with this example that I hadn't noticed. I forgot that Relay modern didn't go to the store.

Ah cool, is that an easy fix? Anything I can help with?

My approach is wrong – I should be doing the caching at the network request level, not at the store level. It's not especially involved.

Fixed in https://github.com/taion/relay-todomvc/commit/180692faf9cb5d1186031ceea6f40149f1e52e7f
cc @staylor

The real answer is that someone open-sources a more robust version of what I wrote, but what I have is enough to illustrate what this demo is supposed to do, I think.

Tried it your fix and it works as expected, thanks! Your solution is interesting, it relies on relay making the same requests, and in the same order, on the server as it does on the client for the initial render. Do you happen to know if that is more or less guaranteed to work with Relay in general, or will that break in some cases you know of?
Also, I'm very curious about what you'd consider a robust/production-ready solution for Relay SSR...

See https://github.com/facebook/relay/issues/1881
There's no reason your requests should go out-of-order though


How to download object from a message?

See the title...
I wan to download the attachment from a message that I received...

I think you can but it's out of scope of this project. See Listen to events do advanced stuff.
And see this for help.


external_location makes run fail because of HIVE_PATH_ALREADY_EXISTS

Hi!
I'm trying to use the "external_location" propery but I get this error when second model runs, as it's trying to use same path again: Target directory for table 'table' already exists.
I've setted the propery on the dbt_project.yml at a project level.
If I set it at a model level, it works the first time but on the second run, i get the same error.
Am i doing something wrong?
For what i've seen, when using the default path, final table path is generated using a UUID but when overriding it, this UUID is not used.
Thanks in advance!

Hey @bmasciarelli,
Apologies for the delay in my response. You are encountering this issue since you are using a static value for external_location. I would advise using a dynamic value if possible (similar to how default path is working with UUID).
Example dynamic values:

Within .sql config block

external_location="s3://dbt-tables/example-directory/" + run_started_at.isoformat()
external_location="s3://dbt-tables/example-directory/" + invocation_id

Within dbt_project.yml

+external_location: "s3://dbt-tables/example-directory/{{ invocation_id }}"

I have updated the README.md to reflect this recommendation so thank you for bring this to my attention 👍
Lastly, I do not automatically append a UUID to a custom external_location value, since I want to give flexiblity to the users of the adapter.
Tomme

No worries Tomme, thanks for the clarification and the amazing work you've done.


<InlineAutocomplete>: Add onSelectSuggestion() callback to <InlineAutocomplete /> to allow execution of additional code after an autocomplete item was selected.

Adds onSelectSuggestion() callback to <InlineAutocomplete /> to allow execution of additional code after an autocomplete item was selected.
Screenshots
Please provide before/after screenshots for any visual changes
Merge checklist

[x] Added/updated tests
[ ] Added/updated documentation
[ ] Changes are SSR compatible
[x] Tested in Chrome
[ ] Tested in Firefox
[ ] Tested in Safari
[ ] Tested in Edge

Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.

@iansan5653 Please note that I removed the code that allowed overwriting of the inserted text. This PR now only adds a callback to notify a consumer what autocomplete item was selected. This should resolve most of the accessibility concerns you've mentioned :)

@iansan5653 accepted your suggestion, thank you :)

Docs updated!


organize shim tests into shims folder, and various test suite tweaks

@bors r+

:pushpin: Commit ed41f1c96980ed26512e10e39406d8c91033a9e7 has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit ed41f1c96980ed26512e10e39406d8c91033a9e7 with merge af033ea428bd469c027d771a9c9fa34485ac495c...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing af033ea428bd469c027d771a9c9fa34485ac495c to master...


fix: allow some cmds to run outside a project

What does this PR do?
Allow the following commands to be run outside an sfdx project:

package installed list
package list
package1 version create

the following ones still require an sfdx project:

package delete
package update
package version displayancestry
package version report
package version delete
package version promote

mostly because the packaging lib expects an SfProject instance to be able to resolve --package being an alias defined in sfdx-project.json.
internal doc: https://salesforce.quip.com/gnuNAtZkcOCP
we haven't decided on this work yet ⬆
What issues does this PR fix or reference?
https://github.com/forcedotcom/cli/discussions/2932
@W-16189175@

packag1 NUT failures are unrelated to these changes, you can see the same errors here:
https://github.com/salesforcecli/plugin-packaging/pull/737

QA:
package list
✅ run outside a project
✅ json match
✅ human match
package installed list
✅ run outside a project
✅ json match
✅ human match
package1:version:create
✅ runs, talks to orgs.  I was getting a Error (1): Package upload operation via API failed. Package not owned by this organization. for all package1 creates


Upload 2.0.1 to cocoapods

Hey ! Could you upload the latest version to cocoapods ?
Cheers

Hi @santiagofm 👋 2.0.1 only contains a fix that is only relevant for the Swift Package Manager, so I didn't bother uploading it. But have done it now 🙂


Headful mode crash on Heroku

Hi there:
I'm facing errors when trying to get my app, of course using Puppeteer, working on Heroku.
By the way, local NodeJS or Heroku (via "heroku local myappname") works perfectly.
As you can see in the following excerpt from the Heroku log, there's an error related to being unable to open X display and then launching the browser:
...
2021-04-12T09:40:17.477766+00:00 app[web.1]: (node:21) UnhandledPromiseRejectionWarning: Error: Failed to launch the browser process!
2021-04-12T09:40:17.477778+00:00 app[web.1]: [32:32:0412/094017.459682:ERROR:browser_main_loop.cc(1386)] Unable to open X display.
2021-04-12T09:40:17.477779+00:00 app[web.1]:
2021-04-12T09:40:17.477779+00:00 app[web.1]:
2021-04-12T09:40:17.477779+00:00 app[web.1]: TROUBLESHOOTING: https://github.com/puppeteer/puppeteer/blob/main/docs/troubleshooting.md
2021-04-12T09:40:17.477779+00:00 app[web.1]:
2021-04-12T09:40:17.477780+00:00 app[web.1]: at onClose (/app/node_modules/puppeteer/lib/cjs/puppeteer/node/BrowserRunner.js:193:20)
2021-04-12T09:40:17.477781+00:00 app[web.1]: at ChildProcess.<anonymous> (/app/node_modules/puppeteer/lib/cjs/puppeteer/node/BrowserRunner.js:184:79)
2021-04-12T09:40:17.477782+00:00 app[web.1]: at ChildProcess.emit (events.js:327:22)
2021-04-12T09:40:17.477783+00:00 app[web.1]: at Process.ChildProcess._handle.onexit (internal/child_process.js:277:12)
2021-04-12T09:40:17.477784+00:00 app[web.1]: (Use `node --trace-warnings ...` to show where the warning was created)
2021-04-12T09:40:17.477840+00:00 app[web.1]: (node:21) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
2021-04-12T09:40:17.477912+00:00 app[web.1]: (node:21) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
...

Any help with this?
Thanks for your time.

I think it would be a better topic on Stack Overflow [puppeteer].
Have you applied everything mentioned in troubleshooting.md#running-puppeteer-on-heroku?
You may miss a dependency, similar errors occurred with CentOS earlier:

https://github.com/puppeteer/puppeteer/blob/main/docs/troubleshooting.md#chrome-headless-doesnt-launch-on-unix
https://stackoverflow.com/questions/60304251/unable-to-open-x-display-when-trying-to-run-google-chrome-on-centos-rhel-7-5


Can't deploy to Kubernetes 1.22: no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1beta1

Describe the bug
I'm getting the following error when I try to deploy to a v1.22 Kubernetes cluster:
zitadelctl takeoff
ts="2021-12-10T17:46:15Z" err="failed to deploy zitadel-operator into k8s-cluster: no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1beta1"" src="mypath/zitadel/cmd/zitadelctl/cmds/root.go:38"

This is because CustomResourceDefinition apiextensions.k8s.io/v1beta1 was removed from the API
I fixed the error by updating the version to v1 in pkg/kubernetes/artifacts.go, for example here.
Afterwards I got the following error:
zitadelctl takeoff
ts="2021-12-10T18:00:13Z" err="failed to deploy zitadel-operator into k8s-cluster: applying crd zitadels.caos.ch failed: CustomResourceDefinition.apiextensions.k8s.io "zitadels.caos.ch" is invalid: [spec.scope: Required value, spec.versions[0].schema.openAPIV3Schema: Required value: schemas are required]" src="mypath/zitadel/cmd/zitadelctl/cmds/root.go:38"

This is because the YAML definitions (here and here) need to be updated to the new spec
Afterwards I was able to deploy the operator.
To Reproduce
See above
Expected behavior
I can deploy to a 1.22 Kubernetes cluster
Would you like me to raise a PR with my updates?

@stebenz @eliobischof

@karfrank I newly generated the CRD-definitions for Kubernetes v1.22, with this there shouldn't be any problems.
I took the freedom myself to create a PR with the changes, so you don't have to.

@karfrank sorry for the delay.
Could you please try it again using zitadelctl version 1.69.5?


Bytecode segfault on Ephemeron test

From #88:

one bytecode test hit a segfault on the Ephemeron test on trunk https://github.com/jmid/multicoretests/actions/runs/3444725852/jobs/5747632136#step:8:851

Originally posted by @jmid in https://github.com/jmid/multicoretests/issues/88#issuecomment-1311983314

The Ephemeron GC-fix was cherry picked to 5.0~trunk 21 days ago: https://github.com/ocaml/ocaml/pull/11631
The Effect/bytecode fix was cherry picked 5 days ago: https://github.com/ocaml/ocaml/pull/11704

I'm wondering whether it is a fresh issue - or caused by one of these that somehow haven't propagated through to install via 5.0.0~trunk and setup-ocaml...

The CI just hit another Ephemeron segfault on 5.0.0~trunk - this one however in native code on Linux:
https://github.com/jmid/multicoretests/actions/runs/3448464623/jobs/5755508518
random seed: 63947905
generated error fail pass / total     time test name

[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test sequential
[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test sequential (generating)
[✓] 1000    0    0 1000 / 1000     0.3s STM Ephemeron test sequential

File "src/ephemeron/dune", line 14, characters 0-107:
14 | (rule
15 |  (alias runtest)
16 |  (package multicoretests)
17 |  (deps stm_tests.exe)
18 |  (action (run ./%{deps} --verbose)))
(cd _build/default/src/ephemeron && ./stm_tests.exe --verbose)
Command got signal SEGV.
[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test parallel

For future reference I include a copy of the relevant CI log for the above Ephemeron 5.0.0~trunk bytecode mode segfault - before it is deleted from the GitHub Actions cache:
random seed: 382251382
generated error fail pass / total     time test name

[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test sequential
[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test sequential (generating)
[✓] 1000    0    0 1000 / 1000     1.9s STM Ephemeron test sequential

[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test parallel
[ ]   22    0    0   22 / 1000    62.6s STM Ephemeron test parallel
[ ]   31    0    0   31 / 1000   132.5s STM Ephemeron test parallel
[ ]   42    0    0   42 / 1000   192.9s STM Ephemeron test parallel
[ ]   44    0    0   44 / 1000   261.6s STM Ephemeron test parallel (shrinking:    3.0002)
[ ]   44    0    0   44 / 1000   322.4s STM Ephemeron test parallel (shrinking:    6.0002)
[ ]   44    0    0   44 / 1000   383.7s STM Ephemeron test parallel (shrinking:    8.0004)
[ ]   44    0    0   44 / 1000   445.1s STM Ephemeron test parallel (shrinking:   15.0004)
File "src/ephemeron/dune", line 14, characters 0-107:
14 | (rule
15 |  (alias runtest)
16 |  (package multicoretests)
17 |  (deps stm_tests.exe)
18 |  (action (run ./%{deps} --verbose)))
(cd _build/default/src/ephemeron && ./stm_tests.exe --verbose)
Command got signal SEGV.
[ ]   44    0    0   44 / 1000   505.9s STM Ephemeron test parallel (shrinking:   28.0006)

Two native code 5.0.0+trunk CI runs for #189 just hit this Ephemeron segfault.
https://github.com/jmid/multicoretests/actions/runs/3461648589/jobs/5779520524
random seed: 256446672
generated error fail pass / total     time test name

[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test sequential
[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test sequential (generating)
[✓] 1000    0    0 1000 / 1000     0.3s STM Ephemeron test sequential

File "src/ephemeron/dune", line 14, characters 0-107:
14 | (rule
15 |  (alias runtest)
16 |  (package multicoretests)
17 |  (deps stm_tests.exe)
18 |  (action (run ./%{deps} --verbose)))
(cd _build/default/src/ephemeron && ./stm_tests.exe --verbose)
Command got signal SEGV.
[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test parallel

and https://github.com/jmid/multicoretests/actions/runs/3461587444/jobs/5779384555
random seed: 296407056
generated error fail pass / total     time test name

[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test sequential
[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test sequential (generating)
[✓] 1000    0    0 1000 / 1000     0.3s STM Ephemeron test sequential

File "src/ephemeron/dune", line 14, characters 0-107:
14 | (rule
15 |  (alias runtest)
16 |  (package multicoretests)
17 |  (deps stm_tests.exe)
18 |  (action (run ./%{deps} --verbose)))
(cd _build/default/src/ephemeron && ./stm_tests.exe --verbose)
[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test parallel

I just checked that the 3 native runs you mention were using an old OCaml compiler. https://github.com/jmid/multicoretests/actions/runs/3461587444/jobs/5779384555#step:4:457 says: The OCaml toplevel, version 5.0.0+dev6-2022-07-21.
This should hopefully be fixed by #195.
The bytecode run, on the other hand, says: 5.0.0+dev8-2022-10-12. I don’t think the log contains the actual upstream commit used, it might still make sense to investigate this one.

Good detective work, sir! :+1:

The native mode Ephemeron segfaults seems to have been fixed with #195
We are still seeing them on 5.0.0+trunk bytecode though:
On the #195 merge CI run: https://github.com/jmid/multicoretests/actions/runs/3477572101/jobs/5813892424
random seed: 385206433
generated error fail pass / total     time test name

[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test sequential
[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test sequential (generating)
[✓] 1000    0    0 1000 / 1000     1.8s STM Ephemeron test sequential

File "src/ephemeron/dune", line 14, characters 0-107:
14 | (rule
15 |  (alias runtest)
16 |  (package multicoretests)
17 |  (deps stm_tests.exe)
18 |  (action (run ./%{deps} --verbose)))
(cd _build/default/src/ephemeron && ./stm_tests.exe --verbose)
Command got signal SEGV.
[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test parallel

On a CI run from PR #197: https://github.com/jmid/multicoretests/actions/runs/3478421458/jobs/5815768968
random seed: 248456036
generated error fail pass / total     time test name

[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test sequential
[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test sequential (generating)
[✓] 1000    0    0 1000 / 1000     2.2s STM Ephemeron test sequential

[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test parallel
[ ]   45    0    0   45 / 1000    74.7s STM Ephemeron test parallel (shrinking:    1)
File "src/ephemeron/dune", line 14, characters 0-107:
14 | (rule
15 |  (alias runtest)
16 |  (package multicoretests)
17 |  (deps stm_tests.exe)
18 |  (action (run ./%{deps} --verbose)))
(cd _build/default/src/ephemeron && ./stm_tests.exe --verbose)
Command got signal SEGV.
[ ]   45    0    0   45 / 1000   135.5s STM Ephemeron test parallel (shrinking:    2)

Maybe you could revert your change of title then ;-)
Found again when trying hard to trigger it:
https://github.com/shym/multicoretests/actions/runs/3480902528/jobs/5821292658#step:11:541
and I still can’t reproduce it locally using the seed obtain in that log :-/

The CI for the Sys PR just hit a 5.0.0+trunk bytecode Ephemeron segfault as well:
https://github.com/jmid/multicoretests/actions/runs/3496455515/jobs/5854385414
random seed: 89312061
generated error fail pass / total     time test name

[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test sequential
[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test sequential (generating)
[✓] 1000    0    0 1000 / 1000     1.9s STM Ephemeron test sequential

[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test parallel
File "src/ephemeron/dune", line 14, characters 0-107:
14 | (rule
15 |  (alias runtest)
16 |  (package multicoretests)
17 |  (deps stm_tests.exe)
18 |  (action (run ./%{deps} --verbose)))
[ ]   12    0    0   12 / 1000   434.7s STM Ephemeron test parallel (shrinking:    0.0002)
random seed: 410265529
generated error fail pass / total     time test name

[ ]    0    0    0    0 / 1000     0.0s STM Float Array test sequential
(cd _build/default/src/ephemeron && ./stm_tests.exe --verbose)
Command got signal SEGV.

After a long bisection, I found it present on all tried commits between trunk and β1 (included): https://github.com/shym/multicoretests/actions/runs/3517413388/jobs/5895155218#step:11:831 but I didn’t manage to get a seed that give reproducibility.

Maybe another issue should be opened, but I indeed saw quite a few issues on native code:

SEGV on Windows: https://github.com/shym/multicoretests/actions/runs/3516624336/jobs/5893420585#step:8:610
Deadlock on native Linux: https://github.com/shym/multicoretests/actions/runs/3516624341/jobs/5893420839 (2 hours without any log)

Another Ephemeron failure on Linux 5.0.0~beta1 bytecode (with some interleaved printing):
https://github.com/jmid/multicoretests/actions/runs/3522096434/jobs/5904675410
random seed: 209315065
generated error fail pass / total     time test name

[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test sequential
[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test sequential (generating)
[✓] 1000    0    0 1000 / 1000     2.5s STM Ephemeron test sequential

[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test parallel
[ ]   19    0    0   19 / 1000    59.2s STM Ephemeron test parallel
[ ]   34    0    0   34 / 1000   129.8s STM Ephemeron test parallel (shrinking:    0.0002)
File "src/ephemeron/dune", line 14, characters 0-107:
14 | (rule
15 |  (alias runtest)
[ ]   34    0    0   34 / 1000   195.0s STM Ephemeron test parallel (shrinking:    3)
random seed: 40688049
generated error fail pass / total     time test name

[ ]    0    0    0    0 / 1000     0.0s STM Float Array test sequential
16 |  (package multicoretests)
17 |  (deps stm_tests.exe)
18 |  (action (run ./%{deps} --verbose)))
(cd _build/default/src/ephemeron && ./stm_tests.exe --verbose)
Command got signal SEGV.

Also seen on Windows 5.0.0+trunk:
https://github.com/jmid/multicoretests/actions/runs/3522215862/jobs/5904929229
random seed: 507906890
generated error fail pass / total     time test name

[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test sequential
[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test sequential (generating)
[✓] 1000    0    0 1000 / 1000     0.5s STM Ephemeron test sequential

[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test parallel
File "src/ephemeron/dune", line 14, characters 0-111:
14 | (rule
15 |  (alias runtest)
16 |  (package multicoretests)
17 |  (deps stm_tests.exe)
18 |  (action (run ./%{deps} --verbose)))

(cd _build/default/src/ephemeron && ./stm_tests.exe --verbose)
Command exited with code -1073741819.
[ ]  106    0    0  106 / 1000    60.9s STM Ephemeron test parallel

Again observed on Linux 5.0.0+trunk bytecode
https://github.com/jmid/multicoretests/actions/runs/3522413690/jobs/5905364853
random seed: 369340211
generated error fail pass / total     time test name

[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test sequential
[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test sequential (generating)
[✓] 1000    0    0 1000 / 1000     2.4s STM Ephemeron test sequential

[ ]    0    0    0    0 / 1000     0.0s STM Ephemeron test parallel
[ ]   35    0    0   35 / 1000    63.0s STM Ephemeron test parallel
[ ]   66    0    0   66 / 1000   163.7s STM Ephemeron test parallel (shrinking:    0.0002)
[ ]   66    0    0   66 / 1000   259.5s STM Ephemeron test parallel (shrinking:    1.0002)
[ ]   66    0    0   66 / 1000   354.3s STM Ephemeron test parallel (shrinking:    1.0004)
[ ]   66    0    0   66 / 1000   449.8s STM Ephemeron test parallel (shrinking:    1.0006)
[ ]   66    0    0   66 / 1000   522.1s STM Ephemeron test parallel (shrinking:    1.0008)
[ ]   66    0    0   66 / 1000   592.4s STM Ephemeron test parallel (shrinking:    2.0002)
[ ]   66    0    0   66 / 1000   663.3s STM Ephemeron test parallel (shrinking:    3)
[ ]   66    0    0   66 / 1000   736.6s STM Ephemeron test parallel (shrinking:    3.0004)
[ ]   66    0    0   66 / 1000   801.2s STM Ephemeron test parallel (shrinking:    4.0002)
[ ]   66    0    0   66 / 1000   874.6s STM Ephemeron test parallel (shrinking:    4.0005)
File "src/ephemeron/dune", line 14, characters 0-107:
14 | (rule
15 |  (alias runtest)
16 |  (package multicoretests)
17 |  (deps stm_tests.exe)
18 |  (action (run ./%{deps} --verbose)))
(cd _build/default/src/ephemeron && ./stm_tests.exe --verbose)
Command got signal SEGV.
[ ]   66    0    0   66 / 1000   936.3s STM Ephemeron test parallel (shrinking:    4.0008)

That's a strange error indeed!
It seems to be failing an assertion on a Field load - which is also what could cause strange behaviour in ephe_get_field underlying the weak issue, so there is a chance/hope that it is the same underlying issue :thinking:

Not spotted since the release of 5.0.0~beta2 with the Weak fix, so closing.


Downgrade Faraday until Octokit is fixed

This can be removed when octokit release a new version with
https://github.com/octokit/octokit.rb/pull/1154

You might be able to resolve this one now.

Thanks @goetzk


ci: check android on ubuntu-22.04

(I guess we could do this to resolve the immediate problem, but it's probably preferable to adjust the android steps to run on 24.04?)

See my comment here, but I think we have a path to staying on easily ubuntu-latest.


Add undo feature (Ctrl+Z)

Ability to "undo" to the last few moves using Ctrl/Cmd + Z

I think allowing a player to undo just one last move can be a nice feature. More than that - doesn't make much sense. This isn't Braid after all.
I'm not sure this is worth updating the original game, though.
@iirelu What do you think?


[FLINK-18131][doc] Add documentation for the new JSON format

#12360  What is the purpose of the change
Add JSON document for new JSON row data format.
Brief change log

Add json.md and json.zh.md

Verifying this change
Build locally and works fine.

The failed tests are not related to this PR, merging...


Update feature gates page

The feature gates was not well sync'ed with the upstream implementation.  This PR resynchronizes the tables based on cross-verification against upstream source code. It also fixes some nits such as a feature was not placed in the right table or the entries are not strictly sorted.

Sounds like a good thing to do.
/approve

/lgtm


#25 Added logic for CSCwb91766: check Subnet and static route overlap

local branch created from https://github.com/jeestr4d/ACI-Pre-Upgrade-Validation-Script/tree/Issue25
for #25

Fix for all comments found in https://github.com/datacenter/ACI-Pre-Upgrade-Validation-Script/pull/129


Add OnRejected to Request Throttling middleware

#10707

Thanks for the submission @Kahbazi ! One of the parts of this middleware I didn't mention clearly is that it is a project our summer intern is working on. However, I think this would be a great opportunity to introduce him to working with external contributors ;).
@DylanDmitri @halter73 Can you both work to see if we can get @Kahbazi 's change iterated on and merged into the repo?
@Kahbazi The first major bit of feedback I can see already is that we're going to need unit tests for this functionality before we could merge it in?

@anurse  I'm often looking for bits that I can contribute, and this middleware seems a good one to me.

it is a project our summer intern is working on

Does this mean that I shouldn't contribute for this middleware?

Does this mean that I shouldn't contribute for this middleware?

You're welcome to contribute but keep two things in mind:

This component is under active development so be careful not to duplicate work (already assigned issues).
We'll have our intern @DylanDmitri review and accept the contributions (with guidance from the rest of us).

I See. Thanks.

Also, the build is complaining about reference assemblies, which is fixed by going to /eng/scripts/ and running .\GenerateRefenceAssemblies.ps1

https://github.com/aspnet/AspNetCore/blob/master/docs/ReferenceAssemblies.md has some docs about how to generate the reference assemblies :). You can actually do it with an MSBuild command.

The MSBuild command was broken for me for a while so I didn't recommend it. It works now though!

Alright @Kahbazi there's one last change and then it's all good to go! This is adding the unit test that Stephen mentioned above.
Throw an exception during the OnRejected callback, and verify that the server replies appropriately.


Parallel video/gif frames upscale

Upscaling videos is slow mainly becasue the frames are upscaled one after the other, this uses only around 10/20% of the system resources, the upscaleGif and upscaleVideo functions could "chunk" the frames and upscale them 2 at a time or 3 (ideally have this as an option with default bring 1 or 2).
I'm doing something similar in my program to upscale more than one file at a time and it runs fine even with 5 files, i'm going to share the code I used, in case it could be of any help, i'm using semophores to have the best possible outcome but it's not necessary and can be just done with arrays and for each
const Semaphore = require("semaphore")

function withSemaphore(sem, f) {
    return new Promise((resolve) => {
        sem.take(() => f().then(sem.leave).then(resolve))
    })
}

let maxFrames = 2
async function upscaleVideo(frames){
  let sem = new Semaphore(maxFrames)
  await Promise.all(frames.map(file => {
      return withSemaphore(sem, () => upscaleImage(file))
  }))
}

async function upscaleImage(img){
    . . .
}

REMEMBER, this does not wait for both frames to finish, but it makes MAX 2 frames upscales at a time (which is even better in theory)
There's also a function to clear the semaphore which can be used to stop upscaling

Also I'm making a video upscale from 480p to 1080p at 30 FPS , clip lasts 40 seconds. Total frames 1200, I'm at 250 frames after 35 mins lol. Adding this feature could save upscale times by A LOT

Also I'm making a video upscale from 480p to 1080p at 30 FPS , clip lasts 40 seconds. Total frames 1200, I'm at 250 frames after 35 mins lol. Adding this feature could save upscale times by A LOT

You can increase the parallel frames with the option parallelFrames (default is 1). I don't recommend setting this too high because your computer will freeze if it runs out of CPU/memory, that's why I didn't make it parallel in the first place. Make sure you are on v0.3.9.


Missing documentation for --query

This command (and others) mentions --query, but the explanation is very simple and there are no examples.

How can I use --query?
What can I do with --query for that I can't do without it?

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

ID: cffe194d-966f-9c21-c7c7-c5470d2bad72
Version Independent ID: 77750236-9104-a3aa-89ad-92bbd7e72ed3
Content: search Command
Content Source: hub/package-manager/winget/search.md
Product: dev-environment
Technology: package-manager
GitHub Login: @KevinLaMS
Microsoft Alias: kevinla

Hey @paulomorgado -
That --query is the default argument in the winget search ...so it doesn't really do anything. It's only needed in the context of using it for an API. I just updated the page to make that more clear -- https://github.com/MicrosoftDocs/windows-uwp/commit/97d6db83b6fc39bbf1240237b9055cd2ee315998
Sorry for the confusion. And thanks for contributing this issue so that it could be improved!


[Refac] Move dataclasses to Pydantic

@smitchaudhary - Can you please provide which use case this might be needed for?

Hey @mlahariya, this is not use case specific. The motivation was that since some of the dataclasses that we use, the likes of, FeatureMapConfig, AnsatzConfig etc need to validate the input in a bunch of different ways in the __post_init__. Pydantic should allow for all this validation with the built in features.


Add a Gitter chat badge to README.md

nodejs/docs now has a Chat Room on Gitter
@bengl has just created a chat room. You can visit it here: https://gitter.im/nodejs/docs.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

LGTM, but I've never really used gitter before. At the worst, this change is super easy to revert.


www/caddy: v1.5.4

Important changes:

[x] Sometimes, Caddy refuses to stop, or to start, especially if the Caddyfile gets big. This change to use the "reload action" allows the use of the admin socket. The rc.d script issues a POST to the REST API of the Caddy admin socket, which gracefully reloads the configuration inside the running process without restarting it. onerestart, onestop and onestart have been renamed to restart, stop and start in the action dictionary of the caddy_control.py

Fixes: https://github.com/opnsense/plugins/issues/3890
Fixes: https://github.com/opnsense/plugins/issues/3887

Minor changes:

[x] Update searchBase() in ReverseProxyController.php for easier maintainability. Requires: https://github.com/opnsense/core/commit/2d45b78f744059089078d56b3c108765b2d23608

[x] Changes in the Model Relation Fields to display options better, everything should be unique now with no double entires. All description fields are now required, since the Model Relation Fields and the internalModelUseSafeDelete require them.

Fixes: https://github.com/opnsense/plugins/issues/3884
Fixes: https://github.com/opnsense/plugins/issues/3885

[x] Move selectpicker empty option to model in general.volt, using BlankDesc. This fixes the option IPv4+IPv6 not appearing in Dynamic DNS.

[x] A custom migration script for this change is not needed, since the Options disappear they default to empty, which is the default that should be.

[x] Comment the Caddyfile template in order to reveal its intricacies for better maintainability.

New Features:

[x] Add the possibility to add a custom HTTP response code in access lists.

Fixes: https://github.com/opnsense/plugins/issues/3883

[x] Add possibility to manipulate multiple headers with "header_up" and "header_down" entries.

[x] Configuration Framework: https://github.com/opnsense/plugins/pull/3891/commits/1da73a8c12df8808ed55bc73b2a4cbb1aed34dda
[x] Caddyfile template logic: https://github.com/opnsense/plugins/pull/3891/commits/1628fd64f9426a9f70c3727c3843c30a09428cca
[x] Template tests: Looks fine, needs some independent testing if possible. It had independant testing: https://github.com/opnsense/plugins/pull/3894#issuecomment-2041631230
Fixes: https://github.com/opnsense/plugins/issues/3881

WIP:
This version is done if nobody finds a bug.

@AdSchellevis This one here is finished. I would like to merge it, since it has a few important fixes. If you have time, could you look over it? Thank you. :)

@Monviech sure :)


fix(README): Validation incorrectly prints "valid"

Btw, thanks for this package. Maybe this is of use to you (or your audience) as well -- I added a link to your repo:
https://gitlab.com/pennersr/djgo/

@pennersr You can submit a PR and add it to README.


When bagging file don't read the whole file into memory

This changes the bagger class so that it doesn't
read the whole file into memory when sending the
file to BagIt. Instead it sends the data a line at a
time.

Coverage decreased (-0.2%) to 63.069% when pulling 93712e0e6a06cf508ccf88b2ae5b7b15524fb7f7 on external-files/perf-improvements into 00772ff26f359c0385f47152e5a78bdae3ed4787 on develop.


Rollup pluggin compatible

Hello,
Thanks for component, very helpful, and congratulations on the last update (1.0.0).
I installed the "rollup" component (https://github.com/rollup/rollup)
I do not know if I did everything right, but I do not think it's compatible with your plugin.
It displays this error message.

Rollup failed: 'Locker' is not exported by node_modules / angular-safeguard / dist / locker.js (imported by
             Src / providers / client.service.ts). For help fixing this error see
             Https://github.com/rollup/rollup/wiki/Troubleshooting#name-is-not-exported-by-module

To configure the component follow these steps:
Https://www.bountysource.com/issues/36339429-feature-request-ionic-2-dev-prod-environment-variables-configuration
Are you able to adapt your component so that it is compatible? Can I wait for this?
Thank you

I'm going to release the code with es6 and es6 builds so it's possible to rollup proper! I'll finish it this weekend

@tuliogallie try this in the meantime https://github.com/rollup/rollup-plugin-commonjs

thank you dude


[WIP] Perfomance testing

Hey Guys.
This is just a POC on performance testing on Hawkular Alerts.
I am using locustio to generate the load  and it is pretty simple to configure with desired load.
The first "example" try to emulate the calls the are made from ManageIQ and this can used to simulate "one manage iq" or more than one.
Please give suggestions/feedback/critics on that PR.
Best Regards,
Guilherme Baufaker

@gbaufake
I would prefer if we align with Hawkular Metrics in tooling and design.
The goal should be to have a similar hook as Hawkular Metrics has and in each PR a CI job can be executed with the desired performance tests.
Also, it should give analysis about potential regressions and an explanation of the scenario.
About tooling and design, I suggest to contact with @FilipB to share their work and align in tooling.
The semantic of the test should take in place the semantics of the operations.
Create definitions is a expensive task and it does not expect to have an excessive load on it, the critical scenarios should be on detection.
Also, performance tests should have context and a well defined environment.
This means executing the load testing tool and the server in the same environment will introduce noise and probably you won't be able to get real info about where a bottleneck exists and why.
So, for 1.x we have a Cassandra process and a Wildfly process, each one should have its JVM limits and we need to have all this info referenced to know how react the software under different parameters, to know where are the limits, and whether a limit reach is a problem or not.


nw builder generate totally incorrect app structure for osx

I'm using nw-builder 3.2.3. After build and sign in i got rejection from app store:
ERROR ITMS-90296: "App sandbox not enabled. The following executables must include the "com.apple.security.app-sandbox" entitlement with a Boolean value of true in the entitlements property list: [( "com.codeorangeinco.raceflight.pkg/Payload/rf.app/Contents/Versions/59.0.3071.115/nwjs Framework.framework/Versions/A/Helpers/crashpad_handler", "com.codeorangeinco.raceflight.pkg/Payload/rf.app/Contents/Versions/59.0.3071.115/nwjs Framework.framework/Versions/A/Resources/app_mode_loader.app/Contents/MacOS/app_mode_loader", "com.codeorangeinco.raceflight.pkg/Payload/rf.app/Contents/Versions/59.0.3071.115/nwjs Framework.framework/Versions/A/XPCServices/AlertNotificationService.xpc/Contents/MacOS/AlertNotificationService" )] Refer to App Sandbox page at https://developer.apple.com/devcenter/mac/app-sandbox/ for more information on sandboxing your app."

Then i tried to sign app from MAS example nw.js app:
http://dl.nwjs.io/v0.19.5-mas-beta/

It works well. Here is apple guide:
https://developer.apple.com/library/content/documentation/MacOSX/Conceptual/BPFrameworks/Concepts/FrameworkAnatomy.html

ah I'm sorry, i think with this all is alright) But app still wont sign:(

According to this issue:
https://stackoverflow.com/questions/45191194/unsealed-contents-present-in-the-root-directory-of-an-embedded-framework/45835046?noredirect=1#comment78714100_45835046
Something wrong with nw builder generated structure, but I can't figure out what exactly:( MAS sign not working

I'm afraid I don't know much about getting an NW.js app into the Mac App Store. Could you tell me what the options you're passing to nw-builder are?

@adam-lynch I spent a week to figure out this. But nothing helped:( https://developer.apple.com/library/content/documentation/MacOSX/Conceptual/BPFrameworks/Concepts/FrameworkAnatomy.html
Try to use MAS if you have developer account
http://docs.nwjs.io/en/latest/For Users/Advanced/Support for Mac App Store/

Ugh I'm sorry.

Try to use MAS if you have developer account

I don't I'm afraid. Your best bet might be to ask here (https://groups.google.com/forum/#!forum/nwjs-general) for suggestions because someone there might have done it.

@Arti3DPlayer let me know if this issue happens with one of the latest versions such as v63 or v64


Social media button after Submitting

Create the ability for users to post to Facebook or Twitter a notice saying,
"I'm voting from my home this November: I just applied online to receive a Virginia absentee ballot"
For Facebook, we can probably create some kind of graphic to accompany it when you are ready to work on this.

eAbsentee.org/confirmation


Training API uses streams

Per discussion in #25, changes here adapt the training API to accept inputs and labels that are streams.

Thanks for putting this together! I left a comment to see if we can address that before we merge. BTW, make sure to run the formatter so it passes the CI!

Ahh I just realized the formatting failure is probably from my recent commit, in any case, feel free to run the formatter on everything so it passes before we merge :)


[ENH] Change _set_fh to a _check_fh that returns self._fh

This implements the change suggested in issue #1776:
Main change is changing the _set_fh(fh) method to _check_fh(fh) which returns self.fh (the coerced/validated ForecastingHorizon), instead of no return.
Further additions concern docstrings which clarify the functionality of _check_fh and the fh attribute.

FYI @kejsitake.
I added you as a potential reviewer, @SveaMeyer13, since this is arising from changes you originally added (see #1776).

@aiwalter, any opinions?

In #1776, @kejsitake said

I agree that this is counterintuitive and _check_fh makes more sense.

so I'll count this as "someone who knows what this actually does and impact" for the purpose of your statement, @chrisholder.
Though I'll give it a few more days so core devs can have a look if they want.

@chrisholder, I had to update this since predict_residuals was added since the PR opened, which had another reference to _set_fh


Crash when loading

https://mclo.gs/98cnacI

Sometimes it happens, sometimes game boots normally

Thank you for bringing this issue to my attention! The problem has been resolved in the latest patch!


Converted the Products,Confirmation,Delivery components into new workflow structure.!

Converted the Products,Confirmation,Delivery components into new workflow structure.!

Looks good to me but could you fix products page, as it shows up bit odd in this PR like shown in the screenshot below;

merged.


Feature: Capture and rethrow errors into the "it"s

It would be nice to be able to do something like the following.
context["do something"] = () =>
{
    int someValue = int.MaxValue;
    MyClass myClass = null;

    beforeEach = () =>
    {
        myClass = new MyClass();
    };

    act = () => myClass.DoSomething(someValue);

    context["happy: value is good"] = () =>
    {
        beforeEach = () =>
        {
            someValue = 1;
        };

        it["will be happy"] = () =>
        {
            myClass.someValue.Should().Be(1);
        };
    };

    context["sad: value is bad"] = () =>
    {
        beforeEach = () =>
        {
            someValue = 256;
        };

        it["will throw an exception"] = expect<ArgumentOutOfRangeException>((exception) =>
        {
            exception.Message.Should().Be("You're an idiot!");
        });
    };
};

For singular cases, maybe something like this:
it["will throw an exception"] = expect<Exception>(() =>
{
    throw new Exception();
},
(exception) =>
{
    exception.Message.Should().Be("You're an idiot!");
});

I'm currently doing the following as a workaround, but a built in feature like above would be nice.
context["do something"] = () =>
{
    int someValue = int.MaxValue;
    MyClass myClass = null;
    bool expectException = false;
    Action action = null;

    beforeEach = () =>
    {
        myClass = new MyClass();
        expectException = false; // have to always set or it will screw up other contexts
    };

    act = () =>
    {
        action = () => myClass.DoSomething(someValue);
        if (!expectException) action();
    };

    context["sad: value is bad"] = () =>
    {
        beforeEach = () =>
        {
            someValue = 256;
        };

        it["will throw an exception"] = () =>
        {
            try
            {
                action();
            }
            catch(ArgumentOutOfRangeException ex)
            {
                ex.Message.Should().Be("You're an idiot!");
            }
        };
    };
};

I can try and find the time to create a PR for this over the next few weeks, but I was REALLY hoping the project had moved on from project.json. I don't look forward to reinstalling 2015.

@BennieCopeland there shouldn't be any "bare" code within the blocks. You need to put
int someValue = int.MaxValue;
MyClass myClass = null;
bool expectException = false;
Action action = null;

into the before.

@amirrajan That fails to build because the variables are now out of scope.

you have to make all your variables members of the test (class level).

That would work, but it could lead to a lot of variables at the class scope. I've never had a problem with scoping inside the contexts. But this is getting away from what this issue is about, which is a feature request for changing the exception handling behavior. Right now, context failures color the tests red (which is great), but I would like it to also hand the tests the caught exception for inspection.

But this is getting away from what the issue is about.

Yep sorry about that. Here is some food for thought: https://github.com/nspec/NSpec/issues/169


recv_from should take a socket addr

recv_from should take a SocketAddr to differentiate it from recv.

Seems invalid


MongoError: not authorized on zenbrain to execute command

I'v just done a stock build and verified mongodb is running (version  v3.4.4) but when I try to run (using the config_sample.js as config.js) I get the following errors:
$ ./run.sh
/home/bitnami/zenbot/node_modules/mongodb/lib/utils.js:123
process.nextTick(function() { throw err; });
^
MongoError: not authorized on zenbrain to execute command { listIndexes: "ticks", cursor: {} }
at Function.MongoError.create (/home/bitnami/zenbot/node_modules/mongodb-core/lib/error.js:31:11)
at queryCallback (/home/bitnami/zenbot/node_modules/mongodb-core/lib/cursor.js:212:36)
at /home/bitnami/zenbot/node_modules/mongodb-core/lib/connection/pool.js:461:18
at _combinedTickCallback (internal/process/next_tick.js:73:7)
at process._tickCallback (internal/process/next_tick.js:104:9)

my guess is in your config, you probably need mongo_username and mongo_password defined.
however, zenbot 3 is not maintained anymore, and likely isn't profitable anymore. 4 will come out soon and you might want to wait for that.


Added Local/Remote Path in config handler to open Vscode in remote ( WSL2 )

if i define new ide_hadlers like vscdode with remote to access wsl2
Example

'vscode_remote' => [
            'handler'        => 'vscode://vscode-remote/',
            'line_separator' => ':',
            'local_path' => 'wsl+Ubuntu/home/username/local/path/in/wsl/nameofproject',
            'remote_path' => '/var/www/html',
        ],

with this configuration lint to file open correctly vscode editor in WSL2 instance ( with remote link )

Ciao @Wit3, grazie per il tuo contributo!
Checks are failing, you must run the fixer and verify commands.
Please have a look at the item 3 of the CONTRIBUTING guidelines.
OPEN QUESTION
If the user chooses vscode-remote when using ds:init, they would still have to open the config file and change username and project name in local and remote path keys.
I think this should trigger a message like:
Please open the config file[link] and configure the directories path for your project`.
Read more at [doc link].

What do you think @luanfreitasdev?

I can't test at the moment with wsl2, I made some changes that dynamically insert the folder name inside the handler.
If all goes well, I'll merge into main.
Thanks

So it worked
 'vscode_remote' => [
        'handler'        => 'vscode://vscode-remote/',
        'line_separator' => ':',
        'local_path'     => 'wsl+' . env('DS_PREFERRED_WSL_DISTRO', 'Ubuntu20.04LTS'),
        'remote_path'    => '/var/www/html',
],


[SPARK-27007][PYTHON]add rawPrediction to OneVsRest in PySpark

What changes were proposed in this pull request?
Add RawPrediction to OneVsRest in PySpark to make it consistent with scala implementation
How was this patch tested?
Add doctest

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8329/
Test PASSed.

Test build #102831 has started for PR 23910 at commit 5f4c895.

Test build #102831 has finished for PR 23910 at commit 5f4c895.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102831/
Test FAILed.

Test build #102836 has started for PR 23910 at commit 0f3a11e.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8332/
Test PASSed.

Test build #102836 has finished for PR 23910 at commit 0f3a11e.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102836/
Test FAILed.

Test build #102838 has started for PR 23910 at commit 458dd65.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8333/
Test PASSed.

Test build #102838 has finished for PR 23910 at commit 458dd65.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102838/
Test PASSed.

Merged build finished. Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8396/
Test PASSed.

Test build #102924 has started for PR 23910 at commit f421ef4.

Test build #102924 has finished for PR 23910 at commit f421ef4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102924/
Test PASSed.

Merged to master


Deploy on Azure

Set up a MySQL database on Azure and connect to it from within application-prod.properties (this needs to be created)

Closing as we won't be using Azure


db-migrator image is taking too much space.

Need to investigate why it's based on mysql:5.6

We won't fix it.  The main reason is requirement to install system level devel package for connecting DB.
The size won't be small unless we customize the installer of the pacakges.


golang version of mwctl w/ corresponding build

The Makefile here is cribbed from blubber
(https://phabricator.wikimedia.org/source/blubber/browse/master/Makefile),
adapted to cross-compile binaries for linux, windows and darwin.  It uses a
locally overridden GOPATH (mwctl/.build) and so when you build, it has to
fetch and build anything that isn't already in GOROOT, which includes the
relevant parts of standard lib for non-native platforms.
TL;DR The non-native builds will take a bit longer to build.  You should use
the apropos make target (i.e. make linux) while iterating development of
mwctl, and then build all to check-in any changes.

Closing in favour of wikimedia/mwctl#1


"Contact" does not redirect the user to a place

Description:
As a user, I can click on the "Contact" option to see the contact information of the website owners.
Actual result:
When I click on this option, I am not redirected to any page or section that contains contact information.
Expected result:
When user clicks on the "Contact" option, they should be given the contact information.
Steps to reproduce:
1- Go to https://www.landver.net/
2- Scroll down until the end of the page
3- Click on "Contact" option
URL: https://www.landver.net/#contact
Screenshot:

hello, I'm Prosper williams. I'm a front-end developer with years of experience. I have experience working with frontend tools like nextJs, typescript, javascript I also have experience working with tailwind CSS. I have gone through your project's folder here on Git Hub and fished out what the problem is, so I'm asking you to let me fix this for you. I'll be sure to work closely with you to know how you want the contact information displayed. I'll send a pull request in 4hours.
thanks.

Hi, @mohsenkhosroanjam
I've checked this on my own.
I don't know, but this seems to be handled already.
The reason for not seeing it is that the contact hashtag is put on a div that is very close to the point where these navigation options are. That's why you aren't seeing it
This is the view for the about section.
I've not checked the code, but it's probably just a CTA to send a mail
However, if you want a special page for contacts, or more things added to the section, then I am willing to handle this issue.

@OWK50GA  Thank you so much for checking and explaining. As an end-user I am confused working with Contact option. I think it is a good idea to change "Newsletter" to "Contact" if it has been supposed to work as a Contact option. What do you think?

@mohsenkhosroanjam
Yes, I think that would be ideal, changing the contact option to "Newsletter". Perhaps even a CTA like "Subscribe to Newsletter", if that's not too long, could also be good.
I'm willing to handle it if necessary 😃

@OWK50GA Seems good. Do I have permission to assign this issue to you? Or I should ask the project owners to assign?

@mohsenkhosroanjam, Sure
Ask the project owners to assign, thank you 😊


Remove unused global styles

this PR alone makes my quick ID look like this:

is this PR dependant on another PR?


After the data in the middle row of the list is modified, the data is moved to the end of the line

OS
Windows
VERSION
1.5.9
ISSUE DESCRIPTION
When I modify the list using the graphical interface of the software, the modified data is placed at the end of the line.
But what I want is to change the data while maintaining the order of the data in the list.
I want to know if I'm using it improperly, or if the underlying logic of the software is just that.
Hope to get some answers, thank you.

update to the latest version


Additional spaces in curried function calls

When using curried functions, this style:
my_func(x)(y)(z)

is cluttered by the parentheses, which resulted in the Sanctuary library adopting this style:
my_func (x) (y) (z)

I request that this spacing be permitted in the whitespace rules for JSLint as it improves readability.

That is not more readable.


Sending too many ably events

I cloned the repository, started the app.. which didnt work, and left my computer. After a couple of hours I started receiving emails that my ably account was blocked because I'd sent over 50k messages. I don't know whats wrong, but thought you should be aware of it.

Hi @vhflat, after you start the game, the server effectively sends 10 messages per second to all the players attached. If you left it without closing, it means that the game was still running and the messages were counting towards your quota, most probably exceeding your free quota (which is why the emails).
Re the app not working, could you tell me what exactly do you mean? What didn't work? It'll be helpful to know and fix anything that needs fixing. I've just checked this again and it's working as expected for me. Here's a deployed version of this project, if you'd like to check it out: https://space-invaders-multiplayer.herokuapp.com/

Closing due to inactivity here.


feat: implement add_language in client

Closes #205

Codecov Report

Merging #207 (4e5410c) into master (eb22d7c) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #207   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            6         6
  Lines          220       222    +2
  Branches        24        24
=========================================
+ Hits           220       222    +2

Impacted Files
Coverage Δ

src/citric/client.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 eb22d7c...4e5410c. Read the comment docs.


I'm not sure

I just wanted you to be able I could just go in your room when I'm in a little hurry up if you're


version 0.10

It looks like that RPC is done differently in 0.10, but is not quite implemented yet. Is that correct?
Could you summarize the principle differences between 0.10 and 0.9.6? It is difficult to see in the code, because of the move of the old source tree.
Thank you!

Yeah, the RPC is being completely redone in 0.10.0, as well as most of the other aspects of the library.  And 0.10.0 isn't ready to go and won't be for at least a few more weeks.
The biggest difference between the older version and what I'm working on now is the schema format support and everything that follows from that.
First off, it's important to note that I'm quite a bit off from having 0.10.0 release ready to go, so these changes are still in flux and what I'm saying here might not represent the final release.
In 0.9.6 and many versions before, you could do a JSON style schema.  In 0.10.0, I'm working to support a schema that is nearly identical to Rust syntax for data types with some additions.  The current working schema format is in this file:
https://github.com/only-cliches/NoProto/blob/master/idl.graphql (I know it has a graphql extension, ignore that)
You'll be able to feed these files into the library, including support for import statements so you can split schema into several files.  You might notice one of the things in the schema file are generics, which has added quite a bit of complexity to the schema and buffer creation process.
Before, you really only had one buffer type per factory, so making a buffer looked like this:
let mut buffer = factory.new_buffer(None);

Now you'll need to do something like this:
let mut buffer = factory.new_buffer("my_type", None);

If the data type you're requesting has generics, you'll need to drop that in as well:
let mut buffer = factory.new_buffer("my_type<u32>", None);

As kind of a natural progression of this, you'll be able spin up custom types on the fly, including custom types:
let mut buffer = factory.new_buffer("(u32, u32, my_type, my_generic_type<u32>)", None);

This adds a great deal of complexity to the new_buffer method.  We've now got to calculate the size of all nested data structures with respect to the generic arguments.
As far as RPC is concerned, I think it's going to just be a data type for the buffers.  Might still have an explicit API to keep things easier, but should look something like this:
// create method call
let mut buffer = factory.rpc_call("my_data_type.some_method", None);
// create method return
let mut buffer = factory.rpc_return("my_data_type.some_method", None);

Those are my current ideas so far, hope all of that makes sense!

Looks very promising.
Another aspect that I have been thinking about is how to use the noproto values in the application code in a more Rusty way that using the get/set methods with the index. Would it make sense to create Rust traits that act as a lazy getter / setter for attributes for the underlying noproto objects? This would mean of course that you must have idea of the types used at compile time, but maintain some flexibitly due to optional nature of the attributes.

Yeah, I completely agree with you that the get and set methods aren't very Rusty!
I had always planned to add macros to this crate at some point, so you could do something similar/identical to serde. The big thing for me is stabilizing the API and the data format so all the work that goes into the macros doesn't get thrown away a few months later.
We'll even be able to do macros with the impl definitions at some point, including being able to generate schemas FROM your macro definitions so they can be used by third parties accessing your API.
I'd like a world where I can update the method name in my rust code on the RPC interface, the method name update gets pushed to my schema file automatically, then a typescript file is built from the schema that can be used to talk to the RPC methods.  We'll get there one day!

That for sure would be fabulous!


Help for Generating Minecraft like Noises

Hi.
How can i generate noise that is same as minecraft terrain noise?
Thanks.

https://notch.tumblr.com/post/3746989361/terrain-generation-part-1

I read it but it is not provides any code, but you can provide a code if understand it.


Move CA src into experimental module

This PR moves the ClimaAtmos src/ into a new module, Experimental, to help avoid name collisions as we move code into src/ (I experienced one with CA's Domains module, which clashes with ClimaCore's Domain structs

bors r+

bors r+


Backend doc-generation PoC.

Implement a hacky solution for backend doc generation and deploy to environment so we can validate the affects it has on performance.

PoC is complete. Updating the findings:
The TLDR is basically that while generating the markdown on the backend removes network bottlenecks, it does not manifest as a significant overall performance improvement.
The reason is that the main bottleneck right now is the pretty rendering of the markdown source itself, which is done using the <ReactMarkdown> component.
To support this statement, we ran two performance analysis using Lighthouse. One for rendering a pretty markdown, and one for rendering it as raw text. Here are the results:
Markdown Format

Full report here: performance-report-eks-markdown.txt (Change .txt to .html)

Raw Format

Full report here: performance-report-eks-raw.txt (Change .txt to .html)

We can see that Time to interactive in the raw case is 0.9s, while in the markdown case its 3.6s.
Both are actually not so bad, and are lower than 5s, which is our current goal.
Even in light of these findings, we are proceeding with the backend migration, for the following reasons:

It's the right thing to do, architecturally. This content is static and should be generated once at ingestion time, regenerating it on every request is wasteful, and may cause more performance issues down the road.
It does provide a substantial improvement for slower internet connection environments (such as 3G mobile).
It makes the front-end much less complicated.
It makes it so we don't have to index all versions of all packages to create documentation (we can just npm install the package on the backend).
It allows running advanced sanitation logic on external inputs (user defined README).
We will have to do some backend pre-processing regardless in order to support packages with a large number of submodules (aws-cdk v2) - so with not a lot of effort, we can go all the way here.
It allows for external consumers to consume the documentation page and embed it in their own site. (aws docs maybe?)

Closing this out.


Ebook intro analogy

Is not quite right, at least for a native English speaker anyways.

The Blockchain is to gold what Bitcoin is to jewelry.

Native English speakers would most likely phrase this as

The Blockchain is to Bitcoin as gold is to jewelry.

BTW, NBitcoin is absolutely amazing. Thank you so much for doing this great work!

Thanks a lot, the book has its own repo on https://github.com/ProgrammingBlockchain/ProgrammingBlockchain .
I will fix it.


Mobile/Only Auto Equipping Poison Rune

I have no clue what changed other than me once again not dealing enough damage to Dysangelos’ second phase in cyan 1* and therefore quitting while I had a poison rune in my hand but since then no matter what level I enter it only auto equips the poison 10* rune.
I’ve tried setting both healthrunes to false, deleting any mention of healthrunes, swapping equipment before and during runs but no matter what I try it immediately swaps back to empty left hand, poison rune 10* right hand. I’ve also cleared the mind stone then done a run, restarted (fully closed the game), done a run then pasted the original unaltered script. There’s been no error messages, even after goring to script to try and get rid of things that might make it equip the poison rune.
Before messing around trying to figure out what broke I’ve only changed things from CONFIG to DEFAULT EQUIPMENT, plus surrounding the brew function call with /* */ to conserve resources.

….. I just tried pasting the original script again and this time it worked. I’m truly at a loss lol.


Add input folder as paremeter.

The current script do not have parameter of the folder containing all the q-value files.

This has been solved in https://github.com/statisticalbiotechnology/mad_decoy/commit/260b026594c45ad0da2acbc67ba9ffd17407f497


Container run and down almost immediately

Describe the problem
I would like to run several instances of Minecraft servers, however, the container starts and stops almost immediately.
configuration :
Debian 11
Docker ce (latest)
4GB RAM
4* CPU on Vmware ESXI
Command used :
docker run -d -it -p 25900:25565 -e EULA=TRUE --name mc-jnf itzg/minecraft-server -v /home/"user"/minecraft-data/jnf:/data
Container definition
Paste run command or compose file here

Container logs
Paste logs here
```[init] Running as uid=1000 gid=1000 with /data as 'drwxrwxr-x 5 1000 1000 4096 Oct 12 15:14 /data'
,[init] Checking for JSON files.
,[init] Resolved version given LATEST into 1.19.2 and major version 1.19
,[init] Resolving type given VANILLA
,[init] Disabling whitelist functionality
,[init] Setting initial memory to 1G and max to 1G
,[init] Starting the Minecraft server...
,Starting net.minecraft.server.Main
,[15:24:26] [ServerMain/ERROR]: Failed to start the minecraft server
,joptsimple.UnrecognizedOptionException: v is not a recognized option
,	at joptsimple.OptionException.unrecognizedOption(OptionException.java:108) ~[jopt-simple-5.0.4.jar:?]
,	at joptsimple.OptionParser.validateOptionCharacters(OptionParser.java:633) ~[jopt-simple-5.0.4.jar:?]
,	at joptsimple.OptionParser.handleShortOptionCluster(OptionParser.java:528) ~[jopt-simple-5.0.4.jar:?]
,	at joptsimple.OptionParser.handleShortOptionToken(OptionParser.java:523) ~[jopt-simple-5.0.4.jar:?]
,	at joptsimple.OptionParserState$2.handleArgument(OptionParserState.java:59) ~[jopt-simple-5.0.4.jar:?]
,	at joptsimple.OptionParser.parse(OptionParser.java:396) ~[jopt-simple-5.0.4.jar:?]
,	at net.minecraft.server.Main.main(SourceFile:84) ~[server-1.19.2.jar:?]
,	at net.minecraft.bundler.Main.lambda$run$0(Main.java:54) ~[?:?]
,	at java.lang.Thread.run(Unknown Source) [?:?]
,2022-10-12T15:24:26.513Z	INFO	mc-server-runner	Done

Your docker command is slightly wrong. You put -v /home/"user"/minecraft-data/jnf:/data after the name of the image (itzg/minecraft-server), which means it's passed to the container (thus the v is not a recognized option) . Your command should look like this:
docker run -d -it -p 25900:25565 -v /home/"user"/minecraft-data/jnf:/data -e EULA=TRUE --name mc-jnf itzg/minecraft-server

(Btw.: latest is not really a helpful version number, since it could mismatch the actual latest within minutes)

Hi, sorry for the late and thanks for the help, it's work !!!


Network Interface panels show to much information

There are too many columns displayed underneath each interface graph.
They frequently break the layout with very long values

Should we split the graphs up into several, smaller graphs?
One for Bytes / Packets / Dropped / Error
That might be a lot of graphs for each interface, and even more with more interfaces,
but it would make handling the values / tooltips a lot easier
Should we drop some information completely?
Should we drop the graphs except for the Bytes sent/received and include the other info as simple statistics?

I would prefer to have the Bytes sent/received as a graph and the dropped information as statistic only.
I think we could drop the packets graph completely or we have a switch where we can switch between packets and byte graph.

I like the idea with the switch, I’ll see what i can come up with

#14 should fix this

Fixed with #14.

Fixed with #14.


fix: add translations for feature events

:tada: This PR is included in version 0.123.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


test(CI): add github action to ensure build succeeds

Cargo generate docs recommend setting up a github action to ensure that your templates build succeeds. I think this is a good thing to have as I was unable to get the regular rust tooling to work without creating a new project from my local clone of this repo.
I mostly copied this build.yml from their docs:
https://cargo-generate.github.io/cargo-generate/templates/authoring.html
but made a few minor changes:

used<EMAIL_ADDRESS>instead of v0.17, as v0.17 does not exist anymore
used dtolnay/rust-toolchain@stable instead of actions-rs/toolchain@v1 as actions-rs is abandoned
made the weekly cronjob run on Saturday instead of Friday
removed **/docs/** from the paths-ignore as no directory matches that for this repo

Here's an example of a failed build when introducing an intentional typo in a template placeholder: https://github.com/loadedice/rust-tui-template/actions/runs/4636065654/jobs/8203626339

hmm, I guess this doesn't really work when making a PR to another repository? Because this project is open source we should get an unlimited amount of hours for free. You can see the CI in action a bit more here but I force pushed a lot of my testing away before submitting this PR :\ https://github.com/loadedice/rust-tui-template/pull/1

Great! I think it will run after we merge the changes.

Yup, all good: https://github.com/tui-rs-revival/rust-tui-template/actions/runs/4645926065/jobs/8222037800


Add macOS building ci/cd

Now GitHub actions have only iOS workflow. Need to add macOS checks in pipeline.

Since we've dropped Carthage support it's irrelevant


Fatal Error when using any Flux Networks Item

I´m playing on my own server and everytime I attempt to open any Flux Network Item I´m kicked from the server with the message "A fatal error has occurred, this connection is terminated" I´ve installed the latest Versions from Flux Networks and Sonar Core for minecraft 1.10.2

Send a crash report and I'll see if I can help, but fyi 1.10.2 is no longer supported.

The problem is the game isn’t crashing. I‘m only kicked from the server and the server log says only Player lost connection

Do you have the debug.log? It should indicate the issue.

I dont have a Debug log for today neither in my .minecraft logs folder nor in my Server log folder

Ah okay, afraid without the log really not much I can recommend.

Ok but thanks for your help


Google reader compatible syncing

Ticket: https://github.com/brentsimmons/NetNewsWire/issues/716
NOTE
This is not ready to merge, see "issues" below. I have hit the point where I could use some suggestions/help.
Overview
Implements support for syncing services that use the Google Reader API (specifically built against FreshRSS).
Includes support for:

Syncing read/unread/starred content
Subscribing to and unsubscribing from feeds
Adding feeds to "folders" (tags)
Removing tags from feeds
Deleting unused tags
Moving feeds to new folders

Issues

OPML import is not really functional. Unlike Feedbin, there is no API for importing OPML, so I was reduced to parsing the file and subscribing to each feed individually. This does not work in practice as the API response is too slow and things time out. Unfortunately I am at a loss as to how to do this better, so I could use suggestions/assistance
Naming. "Google Reader Compatible API" is too long a name. I'd like suggestions for a better public facing description (this will be a bigger issue on iOS where screen space is at a premium)
Icon. I didn't have an icon to represent this. Ideas?
Removing tags (folders). There was a bug (https://github.com/FreshRSS/FreshRSS/issues/2411) in the shipping version of FreshRSS that prevents deleting tags. It appears to have been fixed (https://github.com/FreshRSS/FreshRSS/pull/2412). If you don't have a recent copy, you may need to update.
There is also an oddity in that the API doesn't allow feeds at the top level, so they get dropped into the "Uncategorized" folder. This sometimes causes some oddities.

Testing
You will need a FreshRSS installation to test this (or another Google API server, although I have not tested with any others). Ping me on the NetNewsWire Slack and I may be able to set you up with a test account.

Regarding OPML, I suspect we will have to disable it for some sync services.  I don't think it is widely supported.
Let's just call this Reader API for now.  Maybe we should add a description under the name in the Add account area.
Let's just use a generic RSS icon for now.  I can add it after we merge.