Added landuse=plant_nursery

You've got 9 commits that only end up with a diff changing one file. Did something go wrong with your usage of git?

They submitted #2715 but I didn't pull it all in, instead I squashed it, so their master is out of sync with our master.
@jmespadero can you sync your repository up and try this change again?
A really good way to keep your fork up to date is to add our repository as an upstream remote:
git remote add upstream git@github.com:openstreetmap/iD.git
Then frequently git pull upstream master to keep things in sync.
When you want to make changes, make a feature branch, so that your master branch can always get pulled forward cleanly even if you are working on other things.
git checkout -b my-new-presets
And then do your pull requests from that feature branch.
Another useful tip: if you find yourself making changes and reverting them, do git commit --amend to replace your last commit, instead of making a new one.  It makes it easier for us to see what you did :)
Hope that helps... Feel free to email or IRC me if you have any questions.

Thanks, I have always used SVN, and Git look a bit messy for me.
I have done:
git remote add upstream git@github.com:openstreetmap/iD.git
git pull upstream master
git push
git status
If I compare openstreetmap/iD with jmespadero/iD I still see "10 commits. 1 File change"
I suppose that if I do a pull request, there will be 10 commits again.
So, why are past commits still there?  How do I get rid of them?

It looks like your master branch diverged from ours a while ago... You can try this to sync up:
To preserve your plant-nursery changes someplace:
git checkout -b ahead

To sync up master:
git checkout master
git reset --hard 7b4c6d2
git pull upstream master
git push -f origin

To restore your plant-nursery changes at the end of master branch:
git checkout -b plant-nursery
git cherry-pick c1d3f41 9038c5d
make
git commit --amend
git push -u origin plant-nursery

Ahead branch can be deleted:
git branch -D ahead

Then you should end up with a plant nursery branch that is just 2 commits ahead of our master and redo your pull request from there..
(Lmk if you get stuck, I didn't actually try all of these commands myself)

Git can be kind of confusing at first if you have a background in SVN.  I was that way too..
One thing that really helped me was making this alias in my ~/.gitconfig so that I can run git changes to see the entire history visually.
changes = log --color --graph --pretty=format:'%Cred%h%Creset -%C(yellow)%d%Creset %s %Cgreen(%cr) %C(bold blue)<%an>%Creset' --abbrev-commit

Wow...
I suppose that I could also have done a copy of the plant-nursery.json file elsewhere, sync, and then restored the file.
My problem still is... the make command ends with an error. So when I add a preset, somo files are not updated, and my repository gets out of sync.
If I run make, I don't get much information:
jmespadero@mc ~/bin/iD $ make
node data/maki_sprite.js
make: *** [css/feature-icons.css] Error 1

If I run "make -d",  I got errors related to maki ... which was installed along iD, isn't it?
jmespadero@mc ~/bin/iD $ make -d
 ...

        Trying pattern rule with stem `maki_sprite.js'.
        Rejecting impossible implicit prerequisite `data/maki_sprite.js.S'.
        Trying pattern rule with stem `maki_sprite.js'.
        Trying implicit prerequisite `data/maki_sprite.js.mod'.
        Looking for a rule with intermediate file `data/maki_sprite.js.mod'.
         Avoiding implicit rule recursion.
         Avoiding implicit rule recursion.
         Trying pattern rule with stem `maki_sprite.js.mod'.
         Trying implicit prerequisite `data/maki_sprite.js.mod,v'.
         Trying pattern rule with stem `maki_sprite.js.mod'.
         Trying implicit prerequisite `data/RCS/maki_sprite.js.mod,v'.
         Trying pattern rule with stem `maki_sprite.js.mod'.
         Trying implicit prerequisite `data/RCS/maki_sprite.js.mod'.
         Trying pattern rule with stem `maki_sprite.js.mod'.
         Trying implicit prerequisite `data/s.maki_sprite.js.mod'.
         Trying pattern rule with stem `maki_sprite.js.mod'.
         Trying implicit prerequisite `data/SCCS/s.maki_sprite.js.mod'.
       Trying pattern rule with stem `maki_sprite.js'.
       Rejecting impossible implicit prerequisite `data/maki_sprite.js.c'.
       Trying pattern rule with stem `maki_sprite.js'.
       Rejecting impossible implicit prerequisite `data/maki_sprite.js.cc'.
       Trying pattern rule with stem `maki_sprite.js'.
       Rejecting impossible implicit prerequisite `data/maki_sprite.js.C'.
       Trying pattern rule with stem `maki_sprite.js'.
       Rejecting impossible implicit prerequisite `data/maki_sprite.js.cpp'.
       Trying pattern rule with stem `maki_sprite.js'.
       Rejecting impossible implicit prerequisite `data/maki_sprite.js.p'.
       Trying pattern rule with stem `maki_sprite.js'.
       Rejecting impossible implicit prerequisite `data/maki_sprite.js.f'.
       Trying pattern rule with stem `maki_sprite.js'.
       Rejecting impossible implicit prerequisite `data/maki_sprite.js.F'.
       Trying pattern rule with stem `maki_sprite.js'.
       Rejecting impossible implicit prerequisite `data/maki_sprite.js.r'.
       Trying pattern rule with stem `maki_sprite.js'.
       Rejecting impossible implicit prerequisite `data/maki_sprite.js.s'.
       Trying pattern rule with stem `maki_sprite.js'.
       Rejecting impossible implicit prerequisite `data/maki_sprite.js.S'.
       Trying pattern rule with stem `maki_sprite.js'.
       Rejecting impossible implicit prerequisite `data/maki_sprite.js.mod'.
       Trying pattern rule with stem `maki_sprite.js'.
       Trying implicit prerequisite `data/maki_sprite.js.sh'.
       Looking for a rule with intermediate file `data/maki_sprite.js.sh'.
        Avoiding implicit rule recursion.
        Trying pattern rule with stem `maki_sprite.js.sh'.
        Trying implicit prerequisite `data/maki_sprite.js.sh,v'.
        Trying pattern rule with stem `maki_sprite.js.sh'.
        Trying implicit prerequisite `data/RCS/maki_sprite.js.sh,v'.
        Trying pattern rule with stem `maki_sprite.js.sh'.
        Trying implicit prerequisite `data/RCS/maki_sprite.js.sh'.
        Trying pattern rule with stem `maki_sprite.js.sh'.
        Trying implicit prerequisite `data/s.maki_sprite.js.sh'.
        Trying pattern rule with stem `maki_sprite.js.sh'.
        Trying implicit prerequisite `data/SCCS/s.maki_sprite.js.sh'.
       No implicit rule found for `data/maki_sprite.js'.
       Finished prerequisites of target file `data/maki_sprite.js'.
      No need to remake target `data/maki_sprite.js'.
     Finished prerequisites of target file `css/feature-icons.css'.
     Prerequisite `data/line-icons.json' is newer than target `css/feature-icons.css'.
     Prerequisite `data/relation-icons.json' is newer than target `css/feature-icons.css'.
     Prerequisite `node_modules/maki/www/maki-sprite.json' is older than target `css/feature-icons.css'.
     Prerequisite `dist/img/maki-sprite.png' is newer than target `css/feature-icons.css'.
     Prerequisite `data/maki_sprite.js' is newer than target `css/feature-icons.css'.
    Must remake target `css/feature-icons.css'.
node data/maki_sprite.js
Putting child 0x0122b5a0 (css/feature-icons.css) PID 3882 on the chain.

Solved. I owe you a beer.

I suppose that I could also have done a copy of the plant-nursery.json file elsewhere, sync, and then restored the file.

Haha yes, for just one preset file the git commands are overkill, but maybe next time it will be a bunch of commits that you need preserved.

I suppose that I should add these file also in the commit.

Yes that's right - sorry I forgot the add from the original instructions.
You can just git add . to stage them all..
git add .
git commit --amend


feat(consensus): add central l1 handler transaction

This change is

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#2988  👈 (View in Graphite)
#2921
#2872
#2861
#2715
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


Change input style for search

Can we have border on the search input in the topnav, use 45px height and add What are you looking for? placeholder like this:

not too sure what you want here since it already has a border, but I'll make a PR and we can adjust


failed to compile Firefox

I just used this repo to install firefox and geckodriver but getting this error and I know little about heroku and scripts . Hope this helps.
Installing requirements with pip -----> Firefox geckodriver app detected /tmp/codon/tmp/buildpacks/768374e866288447126d1d75779462919bfbaab4/bin/compile: line 16: cd: /tmp/codon/tmp/buildpacks/768374e866288447126d1d75779462919bfbaab4/fonts: No such file or directory /tmp/codon/tmp/buildpacks/768374e866288447126d1d75779462919bfbaab4/bin/compile: line 82: syntax error near unexpected token ;'
!     Push rejected, failed to compile Firefox
geckodriver app.
!     Push failed`

Pretty much same situation here
-----> Firefox
geckodriver app detected

/tmp/codon/tmp/buildpacks/768374e866288447126d1d75779462919bfbaab4/bin/compile: line 16: cd: /tmp/codon/tmp/buildpacks/768374e866288447126d1d75779462919bfbaab4/fonts: No such file or directory

/tmp/codon/tmp/buildpacks/768374e866288447126d1d75779462919bfbaab4/bin/compile: line 82: syntax error near unexpected token `;'

When will this be fixed?

When will this be fixed?


test: add test to the TableWithBrowserPagination component

fix: #818
Changes proposed in this PR:

add test to the TableWithBrowserPagination component

[ ] I have followed (at least) the PR section of the contributing guide.
@90milesbridge/tigger

Pull Request Test Coverage Report for Build 4290

14 of 14 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 80.705%

Totals

Change from base Build 4241:
0.2%

Covered Lines:
2972

Relevant Lines:
3568

💛  - Coveralls


Disable validation for required props when using :props

I have a case where I need to do something like:
<MyComponent :props={{ @props }}/>

The problem is that MyComponent defines some required props and although the implementation guarantees that @props will contain all required props, the static validator will warn about them since they are not explicitly passed.
The simplest solution would be to automatically disable the static validation whenever :props is used. The downside is that we're losing this validation for all scenarios even though in most cases the set of explicit props and the dynamic props will be mutually exclusive, which would allow the validation to run correctly.
@lnr0626 do you see any alternative?

The only alternative I see right now is supporting something like what we're discussing in #162 with allowing users to specify the structure of a prop - that would allow us to see that @props had the capability of supplying the required props at compile time and disable the warning in that case.


Import warning in pygments used in django

(Original issue 127 created by None on 2006-10-26T21:56:04+00:00)
I use pygments by a BBcode like tag in django and it works but I see this warning in the terminal:
{{{
/usr/lib64/python2.4/site-packages/Pygments-0.1-py2.4.egg/pygments/plugin.py:34:
UserWarning: Module django was already imported from
/usr/lib64/python2.4/site-packages/django/init.pyc, but
/usr/lib64/python2.4/site-packages is being added to sys.path
}}}
Reported by Riklaunim<EMAIL_ADDRESS>

(Original issue was assigned to gbrandl)
(Original issue 127 last updated on 2008-03-05T19:43:03+00:00)
(Issue automaticaly closed due to status in Bitbucket: resolved)

(Original comment by gbrandl on 2008-03-05T19:43:03+00:00)
This is definitely no pygments thing.

(Original comment by None on 2006-10-27T14:48:26+00:00)
birkenfeld verified that with a non django application too:
{{{
#!irc
2006-10-27T14:40:53   ok
2006-10-27T14:41:17   mitsuhiko: #127 comes from setuptools
2006-10-27T14:41:22   something with your entrypoints
2006-10-27T14:42:00   hmm
2006-10-27T14:42:21   birkenfeld: does that happen for you too?
2006-10-27T14:42:44   mitsuhiko: no, but it happens on flying.circus.pocoo.org
}}}
I'll work on that.

(Original comment by None on 2006-10-27T14:38:59+00:00)
Looks more like a django bug to me. Nevertheless I'll have a look at it.


Ability to get/know the type of object for which the factory was called

Greetings!
I have a situation where I want to get an ILogger (from Serilog) dependency in a certain class:
internal sealed class WriteableBitmapPool : IDisposable
{
	...
	public WriteableBitmapPool(ILogger logger)
	{
		_logger = logger;
	}
	...
}

Ideally, the default factory should provide a contextual logger, i.e. it should be obtained from the Log.ForContext<T>() or Log.ForContext(typeof(T)) method, so when WriteableBitmapPool is being created it should get logger instance from result of calling Log.ForContext<WriteableBitmapPool>() method.
Perhaps the API should look something like
private void Setup() => DI.Setup(nameof(Composition))
		...
		.Bind<ILogger>().To(context => Log.ForContext<TSomething>())
		...

where TSomething is replaced with something more suitable
so that the generator can convert this to something like
Serilog.ILogger logger = Log.ForContext<WriteableBitmapPool>();
just before WriteableBitmapPool instance creation
Alternatively add some Type property to IContext so factory could look like
.Bind<ILogger>().To(context => Log.ForContext(context.ObjType))

The problem is that one object can be used as a dependency to create objects of different types. This depends, for example, on the lifetime. I made an example of how I would implement logging.

The problem is that one object can be used as a dependency to create objects of different types. This depends, for example, on the lifetime.

I agree that this is a problem with an ambiguous solution, but in my opinion it can be solved by having each specific ILogger<T> have its own lifetime, i.e. if we register a singleton ILogger<T> that can be obtained by requesting ILogger in the constructor, all objects of the Dependency type will receive their single instance of Logger<Dependency>, and all objects of the Service type will receive their single instance of Logger<Service>. I believe this is how generic Dependency<TT> work.

I made an example of how I would implement logging.

The solution with creating additional types representing loggers for specific types certainly has a place to be, but it creates unnecessary verbosity and problems in cases where, for example, it is necessary to pass a non-default logger.
Also, apparently, in this example there is a composition configuration error (it breaks off in mid-sentence)

@Neakita could you please review this approach?

@NikolayPianikov
This is perfect for my case, however, the choice of name and the fact that the meaning depends on lifetime can be confusing, although I can't think of something more appropriate.
Unless, judging by the name, the dependency is given to the object in ownership, which I would consider a bad approach, because then it should take responsibility for managing the lifetime of its dependencies, which it can only do if it creates them itself, although in this case it gets them from outside and the owner is considered a Composition that should manage their lifetime, there is an inconsistency here when the lifetime is, for example, transient.

@Neakita Yes that's a reasonable point. Please see this option.

@NikolayPianikov
As far as I understand, in this example you replaced OwnerType with ConsumerTypes, where in the case of Singleton or PerResolve dependency there will be several types that use it.
It seems to me that the previous example providing the OwnerType property was more understandable and easier to use, whereas the option with ConsumerTypes looks more flexible, I can't think of ways to use it at the moment. However, it seems to me that the Consumer looks more appropriate naming than the Owner.
My point was that, as you said in your report on DotNext, Pure.DI is intentionally not responsible for the lifetime of dependencies of the Transient type implementing IDisposable because it requires a lot of overhead.
I have a suggestion about this that I would like to make separate issue later.
In my opinion, something like ConsumerType will work more than adequately, however, for such a scenario, open generics can also be adapted for grouping by type, so Log.ForContext<TT> can be a singleton for each concrete TT, although I suppose it is too difficult to implement for such a narrow use case, but it will also worsen performance because it will be necessary to use lock for all these resolves, additional time will be spent on compiling all generic methods, and objects will not be able to be collected by GC until Composition is disposed.
I'm sorry if I'm confusing or distracting you.

As far as I understand, in this example you replaced OwnerType with ConsumerTypes, where in the case of Singleton or PerResolve dependency there will be several types that use it.

Yes, that's the way it is

It seems to me that the previous example providing the OwnerType property was more understandable and easier to use, whereas the option with ConsumerTypes looks more flexible, I can't think of ways to use it at the moment. However, it seems to me that the Consumer looks more appropriate naming than the Owner.

The array option would be useful when we want to account for all consumers.

My point was that, as you said in your report on DotNext, Pure.DI is intentionally not responsible for the lifetime of dependencies of the Transient type implementing IDisposable because it requires a lot of overhead.

Right now it is possible to control transient IDisposable. See this or this example.

In my opinion, something like ConsumerType will work more than adequately, however, for such a scenario, open generics can also be adapted for grouping by type, so Log.ForContext<TT> can be a singleton for each concrete TT, although I suppose it is too difficult to implement for such a narrow use case, but it will also worsen performance because it will be necessary to use lock for all these resolves, additional time will be spent on compiling all generic methods, and objects will not be able to be collected by GC until Composition is disposed.

I haven't figured out how to use context to call a method with a generic type.

I haven't figured out how to use context to call a method with a generic type.

Not a context, but marker type such as TT, TT1, etc, some brand new TConsumer.
But again, if OwnerType/ConsumerTypes/ConsumerType will be implemented, in my case I would prefer to use it because Log.ForContext<T>() under the hood still calls Log.ForContext(typeof(T)), however, a certain TConsumer marker type can also be potentially useful.

The array option would be useful when we want to account for all consumers.

Not a logger case, maybe something other. I saw the generated code and it can actually be stackalloc ReadOnlySpan which prevents unnecessary array allocation.

Not a logger case, maybe something other. I saw the generated code and it can actually be stackalloc ReadOnlySpan which prevents unnecessary array allocation.

This is already implemented for some cases, e.g. see here.

This is already implemented for some cases, e.g. see here.

I'm talking about this.

Not a context, but marker type such as TT, TT1, etc, some brand new TConsumer. But again, if OwnerType/ConsumerTypes/ConsumerType will be implemented, in my case I would prefer to use it because Log.ForContext<T>() under the hood still calls Log.ForContext(typeof(T)), however, a certain TConsumer marker type can also be potentially useful.

Yeah I get your idea, created a ticket and will think about it.

Fixed in 2.1.42


Fix missing required field

I add channelId is not specified but is required

this needs a format change
check activity.md for example

Can you read i change

usage is wrong
$reactionCollector[channelId;messageId;userFilters;time;reactions;awaits;removeReaction?; awaitData?;endAwait?]

And now is good or need to update example


serialComma usage

Hello,
Is it possible to remove all the commas, in the humanized string?
Regards

Does the delimiter option work?

It does indeed. Thank you for pointing it out.


add dat-json into dat-node

now dat.meta.read and dat.meta.write are available to deal with the dat.json file in the repository.

weird, why does the pr fail but the push doesnt? @joehand

weird, why does the pr fail but the push doesnt? @joehand

Hmm, its some pesky peer count tests. I thought I put enough leeway into them but I guess I need some more. I ran it again and its passing but I'll look at those tests again.
Thanks for doing this! Can we add some tests and docs for this too?

Codecov Report

Merging #122 into master will decrease coverage by 4.48%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master     #122      +/-   ##
==========================================
- Coverage   98.23%   93.75%   -4.49%
==========================================
  Files           6        7       +1
  Lines         283      304      +21
==========================================
+ Hits          278      285       +7
- Misses          5       19      +14

Impacted Files
Coverage Δ

dat.js
98.95% <100%> (+0.02%)
:white_check_mark:

lib/dat-json.js
26.31% <26.31%> (ø)

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 be513b3...1629563. Read the comment docs.

I guess this would also need a dat.meta.write() on archive creation?


Project Level 8_4: self-check 3 not made available after done serving

Summary
Hello all, has anyone else run into the same problem for test case 8_4? it seems that my self-check 3 is unavailable and does not serve customer 12 after it is done serving customer 9.
I implemented self-check as an extension of Server and hence should retain the same functionality when being updated through the various execute methods. Thanks in advance!
Screenshots (if any):

for my selfcheckout server, after it has done serving the current customer, it will check if it has any waiting customers available. if have, it will serve the next latest waiting customer.

for self-check out, using the share queue, check which has the latest service timing and pass the serve to the serve event

Hello,
It is likely your customer 12 & 13 which is waiting at self-check 2 is only waiting for self-check 2 to become available again. Since self-check shares one queue, these customer should wait for the next available self-check regardless if the serverID is the same as the one the customer is waiting at.

You can add some System.out.println(...) in between your code to check what actually is the cause to this problem. For me it seems like your Customer 12 and 13 is only waiting at SelfCheck2 while it should be all the SelfChecks. If this is the problem you can try to first check if this current server is a SelfCheck, if it is then you can retrieve all the SelfCheck from your Shop's server list and find the SelfCheck with the shortest "waiting time"(i.e. next available time), and use that SelfCheck instead of the one that you have returned from the previous event(which I assumed to be SelfCheck 2).

you should have a check after every done event if there are any customers in the queue. if so, there should be a new serve event created.

Thank you everyone! I realised that I didnt update my SelfCheckout to take serve the next SelfCheckout with the earliest done time so I iterated through my shop to find that server.


Update golang.org/x/text to 0.3.8

Even if auditevent is not vulnerable to CVE-2022-32149, it's good to
stay up to date.
Signed-off-by: Juan Antonio Osorio<EMAIL_ADDRESS>

Codecov Report

Merging #113 (ac2f710) into main (d0a41b0) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #113   +/-   ##
=======================================
  Coverage   97.04%   97.04%
=======================================
  Files          10       10
  Lines         338      338
=======================================
  Hits          328      328
  Misses          7        7
  Partials        3        3

Flag
Coverage Δ

unittests
97.04% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Update lib and fix gemspec

Fixes #23 and will now support Raspberry Pi 2 (tested).

Thanks! I'll get this reviewed and merged.

@jwhitehorn it would be great if you could merge this faster. I've got a lot of possible improvements in mind for the gem.


About Blob

Hi All and Shen2
I downloaded this library months ago and have been using it since then. I decided to get new updates. Everything seems working well except blob column values returns NULL.
I realized that getValue function in Type\Blob.php returns NULL since the blob data is in $this->_binary variable not in $this->_value.
ORJINAL FUCNTION which returns NULL:
public function getValue(){
return $this->_value;
}
AFTER CHANGING (returns actual blob data):
public function getValue(){
return $this->_binary;
}
What is wrong here ? Am I wrong ?
Thanks in advance

This bug has been fixed in https://github.com/duoshuo/php-cassandra/commit/950f0070d81c80a300eaf4960c41145545ff3e44
Can you have a try and tell me whether it works?


Problem with "Manipulating Arrays With unshift" page

There seems to be an extra closing bracket in the suggested answer on the left panel: [["Paul", 35], ["dog", 3]]).

@ltegman Am I blind that I can't see this?  These are multi-dimensional arrays, are they not?

Closing bracket is probably the wrong wording. There's a straggling paren at the end of the array that shouldn't be there.

I thought I was going crazy too until I saw the parenthesis.


Deploying on serverless causes error with fetch not defined

So I am following this doc: https://docs.newrelic.com/docs/serverless-function-monitoring/aws-lambda-monitoring/get-started/enable-serverless-monitoring-aws-lambda#connect-aws. So I can turn on monitoring for my lambda function.  I follow all the instructions and for  Enable Lambda instrumentation step 3 i use option 1 for the serverless plugin.  I follow the instructions and then when i go to deploy i get the error:
Serverless: WARNING: Function api has timeout of 60 seconds, however, it's attached to API Gateway so it's automatically limited to 30 seconds.
		  Reference Error ----------------------------------------

		  ReferenceError: fetch is not defined
		      at /home/runner/work/microservice_repair/microservice_repair/node_modules/serverless-newrelic-lambda-layers/dist/api.js:16:17
		      at Generator.next (<anonymous>)
		      at /home/runner/work/microservice_repair/microservice_repair/node_modules/serverless-newrelic-lambda-layers/dist/api.js:8:71
		      at new Promise (<anonymous>)
		      at __awaiter (/home/runner/work/microservice_repair/microservice_repair/node_modules/serverless-newrelic-lambda-layers/dist/api.js:4:12)
		      at Object.exports.nerdgraphFetch (/home/runner/work/microservice_repair/microservice_repair/node_modules/serverless-newrelic-lambda-layers/dist/api.js:12:53)
		      at NewRelicLambdaLayerPlugin.<anonymous> (/home/runner/work/microservice_repair/microservice_repair/node_modules/serverless-newrelic-lambda-layers/dist/integration.js:26:49)
			  at Generator.next (<anonymous>)
			       at /home/runner/work/microservice_repair/microservice_repair/node_modules/serverless-newrelic-lambda-layers/dist/integration.js:8:71
			       at new Promise (<anonymous>)
			       at __awaiter (/home/runner/work/microservice_repair/microservice_repair/node_modules/serverless-newrelic-lambda-layers/dist/integration.js:4:12)
			       at NewRelicLambdaLayerPlugin.check (/home/runner/work/microservice_repair/microservice_repair/node_modules/serverless-newrelic-lambda-layers/dist/integration.js:23:16)
			       at BbPromise.reduce (/opt/hostedtoolcache/node/10.22.0/x64/lib/node_modules/serverless/lib/classes/PluginManager.js:498:55)
			   From previous event:
			       at PluginManager.invoke (/opt/hostedtoolcache/node/10.22.0/x64/lib/node_modules/serverless/lib/classes/PluginManager.js:498:22)
			       at getHooks.reduce.then (/opt/hostedtoolcache/node/10.22.0/x64/lib/node_modules/serverless/lib/classes/PluginManager.js:533:24)
			   From previous event:
			       at PluginManager.run (/opt/hostedtoolcache/node/10.22.0/x64/lib/node_modules/serverless/lib/classes/PluginManager.js:533:8)
			       at variables.populateService.then (/opt/hostedtoolcache/node/10.22.0/x64/lib/node_modules/serverless/lib/Serverless.js:168:33)
			   From previous event:
			       at Serverless.run (/opt/hostedtoolcache/node/10.22.0/x64/lib/node_modules/serverless/lib/Serverless.js:155:74)
			       at serverless.init.then.then (/opt/hostedtoolcache/node/10.22.0/x64/lib/node_modules/serverless/scripts/serverless.js:50:26)
			       at runCallback (timers.js:705:18)
			       at tryOnImmediate (timers.js:676:5)
			       at processImmediate (timers.js:658:5)
			       at process.topLevelDomainCallback (domain.js:126:23)
			   From previous event:
			   at Object.<anonymous> (/opt/hostedtoolcache/node/10.22.0/x64/lib/node_modules/serverless/scripts/serverless.js:50:4)
			        at Module._compile (internal/modules/cjs/loader.js:778:30)
			        at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)
			        at Module.load (internal/modules/cjs/loader.js:653:32)
			        at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
			        at Function.Module._load (internal/modules/cjs/loader.js:585:3)
			        at Module.require (internal/modules/cjs/loader.js:692:17)
			        at require (internal/modules/cjs/helpers.js:25:18)
			        at Object.<anonymous> (/opt/hostedtoolcache/node/10.22.0/x64/lib/node_modules/serverless/bin/serverless.js:47:1)
			        at Module._compile (internal/modules/cjs/loader.js:778:30)
			        at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)
			        at Module.load (internal/modules/cjs/loader.js:653:32)
			        at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
			        at Function.Module._load (internal/modules/cjs/loader.js:585:3)
			        at Function.Module.runMain (internal/modules/cjs/loader.js:831:12)
			        at startup (internal/bootstrap/node.js:283:19)
			        at bootstrapNodeJSCore (internal/bootstrap/node.js:623:3)

Tried older version but still same error.  Any thoughts on what I am missing?

@svarughese91 Thanks for reporting. Are you getting this error with v0.1.x?

v0.2.2

@svarughese91 Please  upgrade to v0.2.3 and let us know if you continue to see errors. Thanks again.


Currently we continue estimating the fee even if an error occurs. This PR refactors the estimate fee function to use map and into_iter to stop execution on error.

@lana-shanghai starknet foundry is failing


Fix login issue with undefined user

There was an issue where the user already existed in the DB so not self.pk was not triggering during a user save. This gets around that by checking the current stack to see we are in a User save from the authenticate method.

I'm just going to pull back and do the rest of my comments up here at the top level, since everything that's left basically boils down to the same thing.

I very strongly believe you should not be doing high-level business logic checks of "did a developer leave a view unguarded?" in a very low-level method like save().
I think at this point that both None and AnonymousUser should be converted into User(username='_system'), if we insist on having the created_by and updated_by fks be non-nullable.  Both cases mean that system-level logic (in the AnonymousUser case, authentication logic) created or updated the object.

That said, your logic on line 102 is incorrect -- you are relying on the validation method to catch the AnonymousUser that will (almost certainly) be falling through that line and stored into user.

I think something like:
user = get_current_user()
if user is None or not user.is_authenticated():
    user = get_system_user()

and then do away with the impersonate() and _check_user().


Update thesaurus wordlist URL

According to this, the new URL in this commit "includes the ox.ac.uk archive and more".
Fixes #629

Codecov Report

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

@@            Coverage Diff             @@
##           master    #3583      +/-   ##
==========================================
+ Coverage   73.48%   77.31%   +3.82%
==========================================
  Files          99       99
  Lines      138497   139177     +680
==========================================
+ Hits       101776   107599    +5823
+ Misses      36721    31578    -5143

Impacted Files
Coverage Δ

src/gui_beval.c
55.02% <0%> (-0.88%)
:arrow_down:

src/xxd/xxd.c
58.49% <0%> (+0.02%)
:arrow_up:

src/if_tcl.c
86.42% <0%> (+0.54%)
:arrow_up:

src/if_ruby.c
92.3% <0%> (+0.8%)
:arrow_up:

src/beval.c
3.6% <0%> (+0.9%)
:arrow_up:

src/if_lua.c
85.28% <0%> (+1.19%)
:arrow_up:

src/regexp_nfa.c
91.37% <0%> (+1.23%)
:arrow_up:

src/main.c
63.87% <0%> (+1.3%)
:arrow_up:

src/if_perl.xs
86.12% <0%> (+1.34%)
:arrow_up:

src/tag.c
72.43% <0%> (+1.46%)
:arrow_up:

... and 81 more

Continue to review full report at Codecov.

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

I'll include it, thanks!


ImportError: No module named 'future`

from future import standard_library
I am using anaconda3. future seems to be installed (checked with pip). Any how to get through this?

In that case future isn't installed properly on your system. Can you list any steps to reproduce this?

I just installed anaconda3.
pip install future
shows
Requirement already satisfied (use --upgrade to upgrade): future in /home/immanuel/programs/anaconda3/lib/python3.4/site-packages/future-0.15.2-py3.4.egg

clean install cleared it


Feature Request: Enhance Scheduled Task Options

I have done some searching and not seen this mentioned before.
Request: More Options for Scheduled Task abilities, specifically:

Move/Rename/Delete Files/Folders From that specific AMP Instance OR the entire AMP directory

Example
Using an existing Schedule, Add a New Task named FileMovement
Input: Select Dropdown (Copy File, Delete File, Move File, Copy Directory, Delete Directory, Move Directory)
- Copy File
- From Path : Root Path = instance folder if used from the specific instance or AMP path if used from the main AMP Page (Ex: (Root Instance Folder)/Minecraft/world/data/file.text)
- To Path: Root Path = instance folder if used from the specific instance or AMP path if used from the main AMP Page  (Ex: (Root Instance Folder)/Minecraft/world/data/backup/file.text)
- Delete File
- From Path : Root Path = instance folder if used from the specific instance or AMP path if used from the main AMP Page (Ex: (Root Instance Folder)/Minecraft/world/data/file.text)
- Move File
- From Path : Root Path = instance folder if used from the specific instance or AMP path if used from the main AMP Page (Ex: (Root Instance Folder)/Minecraft/world/data/file.text)
- To Path: Root Path = instance folder if used from the specific instance or AMP path if used from the main AMP Page (Ex: (Root Instance Folder)/Minecraft/world/data/file.text)
- Copy Directory
- From Path : Root Path = instance folder if used from the specific instance or AMP path if used from the main AMP Page (Ex: (Root AMP Folder)/Instance01/Minecraft/world/data/)
- To Path: Root Path = instance folder if used from the specific instance or AMP path if used from the main AMP Page (Ex: (Root AMP Folder)/Instance02/Minecraft/world/data/backup/)
- Delete Directory
- From Path : Root Path = instance folder if used from the specific instance or AMP path if used from the main AMP Page (Ex: (Root AMP Folder)/Instance01/Minecraft/world/data/)
- Move Directory
- From Path : Root Path = instance folder if used from the specific instance or AMP path if used from the main AMP Page (Ex: (Root AMP Folder)/Instance01/Minecraft/world/data/)
- To Path: Root Path = instance folder if used from the specific instance or AMP path if used from the main AMP Page (Ex: (Root AMP Folder)/Instance02/Minecraft/world/data/backup/)

It would be great if it's an option to enter the time schedule manually in CRON format instead of having to click around in the GUI.


HMS-3187 bump: dependabot violations

Add the suggested changes from dependabot.
  Waiting for CI.


bugfix: fix resizeable column coords when serverside true

Should close issue #171

Coverage decreased (-0.2%) to 72.851% when pulling 957d3241d22b882e9ee54e9529d9a2b631efb01d on bugfix/serverside-resize-bug into 30e4e22cfba7f7f82da80fa5c3d5874aa51211f1 on master.


Add F# version

Make the koans available in F#.
Some questions to answer :

do we use the same concepts (EventPublisher) ?

Avoid EventPublisher to be more functional oriented -> return events

It's done !


Open_Core0 Config

This is going to sit for a few while I wait for a new board version to arrive.

OpenCore0 merged in another PR


style: change cell count chart's icon type

Legends on the following pages can be updated too to keep them consistent in style

https://explorer.nervos.org/charts/average-block-time
https://explorer.nervos.org/charts/difficulty-hash-rate
https://explorer.nervos.org/charts/total-dao-deposit
https://explorer.nervos.org/charts/new-dao-deposit
https://explorer.nervos.org/charts/inflation-rate
https://explorer.nervos.org/charts/secondary-issuance
https://explorer.nervos.org/charts/total-supply

Legends on the following pages can be updated too to keep them consistent in style

https://explorer.nervos.org/charts/average-block-time
https://explorer.nervos.org/charts/difficulty-hash-rate
https://explorer.nervos.org/charts/total-dao-deposit
https://explorer.nervos.org/charts/new-dao-deposit
https://explorer.nervos.org/charts/inflation-rate
https://explorer.nervos.org/charts/secondary-issuance
https://explorer.nervos.org/charts/total-supply

All legend icon change to  'roundRect'


Incomplete Opengl Interface

Describe the bug
Some interface in Opengl cannot use in pyglet on my computer.
Also, I cannot find a pyglet.gl.glu package.
This is my code:
from pyglet.gl import *
glMatrixMode(GL_PROJECTION)
glLoadIdentity()
gluPerspective(65, width / height, 0.1, 60)
glMatrixMode(GL_MODELVIEW)
glLoadIdentity()

All of the function I used is not defined when I try to run.
System Information:

Platform: 2020 Macbook Pro, M1 8 cores, 8G RAM
System: MacOS Ventura 13.1
Python Version: Python 3.10
Pyglet Version: pyglet 2.0.4

You should pin your pyglet version to 1.5.x if you want to use the fixed pipeline. Pyglet 2 is using OpenGL 3.3 core were the fixed pipeline no longer exist. We do have a gl_compat module exposing functions and enums from a compatibility context, but there are no GLU.

I can get all functions from gl_compat now, but gluPerspective is still missing. I also try to use pyglet 1.5, which contain gluPerspective, but it seems that it does not support python 3.10. It gives out a error:
AttributeError: dlsym(0x3e357a420, class_getMethodImplementation_stret): symbol not found. Did you mean: 'class_getMethodImplementation'?

Yes, there are no GLU in pyglet 2.0. You're not really supposed to mix fixed and programmable pipeline there unless you know exactly the consequences of that. (How to handle bound shaders and vertex arrays to fall back to the fixed pipleine etc)
More info in migration guide if you are planning to jump over to 2.0 later : https://pyglet.readthedocs.io/en/latest/programming_guide/migration.html
For the other error you should probably make a new issue speciying the exact pyglet version, your operating system and sharing the entire stack trace, not just the error itself.

Closing this because the lack of activity. Be free to open new issue or comment further on this one (reopen)


Move ETL close to WordCount

This is preparing for promoting ETL to a core exercise. As a core, ETL should go just after Word Count. Let's first check if, as a side,  it isn't too hard on this point in the track by making it unlocked by WordCount.
The difficulty score is arbitrary; the Track Anatomy Project didn't touch those yet. I just made it one step more difficult than WordCount.

@exercism/ruby

The changes look good.  However the Readme suggests that it should be after the Scrabble game is done... otherwise it isn't really indicating the historical truth.

That's an interesting observation @kotp.
For reference, here's the relevant part of the description:
We're going to extract some scrabble scores from a legacy system.

The old system stored a list of letters per score:

1 point: "A", "E", "I", "O", "U", "L", "N", "R", "S", "T",
2 points: "D", "G",
...
The shiny new scrabble system instead stores the score per letter, which makes it much faster and easier to calculate the score for a word. It also stores the letters in lower-case regardless of the case of the input letters:

"a" is worth 1 point.
"b" is worth 3 points.
....

Your mission, should you choose to accept it, is to transform the legacy data format to the shiny new format.

I did not interpret this as if the legacy was the system from a preceding exercise. On the contrary, I read it as inspiration to use the second format for the Scrabble Score exercise :-)
Do we need to take action as follow up?
(I'm merging this for now, but please let me know if we need to follow up.)

I think if the only reference to "a legacy system" is indicating "the old system" on the following line, then the old system would be our prior Scrabble solution?  If I had seen "An old system" it would have indicated to me some system, rather than the one that we may be familiar with.
It may be due to having done "the old system" in a specific way that we are reminded of that old system specifically.

There may be nothing to do here.  If it is before any other Scrabble exercises, then it will automatically not be referring to that, as it hasn't happened yet.


[incubator/kafka] Wrong indentation fixed

Labelling PR from yesterday (#17392) introduced a bug on indentation. Sorry, this should fix it.

/ok-to-test


skypeforlinux: <IP_ADDRESS> -> <IP_ADDRESS>

Motivation for this change
Version bump from <IP_ADDRESS> -> <IP_ADDRESS>
Things done

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

[X] NixOS
~[ ] macOS~
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Assured whether relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Thanks :+1:


AdonisUI-Demo main Window falls back to default WPF not inheriting Adonis Windows Style.

Describe the bug
The code compiles and when AdonisUI.Demo is run the main windows doesn't inherit from Adonis and falls back to default WPF Window. Editing the MainWindow.xaml throws
Severity	Code	Description	Project	File	Line	Suppression State Error		'SpaceExtension' is not valid for Setter.Value. The only supported MarkupExtension types are DynamicResourceExtension and BindingBase or derived types.
AdonisUI.Demo	MainWindow.xaml
To Reproduce
Steps to reproduce the behavior:

Clone Repository
Build run
Edit MainWindow.xaml
See error

Expected behavior
MainWindow inherits the Adonis-UI theme.
Screenshot

Additional context
dotnet --version
2.2.203 Windows 7 x64

I am not sure if I understand you correctly. To me this looks like an issue with the XAML Designer but you said it happens "when AdonisUI.Demo is run". Could you please clarify if the error occurs in the Visual Studio Designer or at runtime in the application?
Regarding the error you received:
Using the designer I can reproduce the error in some situations. It occurs because the designer cannot handle the usage of MarkupExtensions in style setters. It works for some hard-coded types like Binding and DynamicResource but custom extensions like the SpaceExtension do always raise this error. As this is not the first case where the designer has problems dealing with the SpaceExtension, I will try to find an alternative that does not rely on custom MarkupExtensions.
Regarding the styles not being applied in the designer:
Besides the error, the designer seems to fail to load resources that reside in App.xaml. This is not an issue with the library though. If you want the designer to make use of the styles you can include them explicitly in the window's resources in addition to the app's resources. For me, the designer is able to find the resources in that case. This might have performance implications though as the ResourceDictionaries are instantiated multiple times in that case. So my advice would be to just accept that some styles might not be applied in the designer as it is not a perfect copy of the application anyways.

The error only occurs in XAML Designer. The app works in run time except not inheriting the Window style. But now your comments clarified to me where I went wrong. I close the issue now.


Feature add variant_cast

Added global "variant_cast" method
Added 4 overloads of the method "variant_cast"
In order to move values out, to return values by reference
or to return a pointer to the underlying values, when the type does match:
template
T variant_cast(variant&& operand);
template
T variant_cast(variant& operand);
template
T variant_cast(const variant& operand);
template
const T* variant_cast(const variant* operand) RTTR_NOEXCEPT;
template
T* variant_cast(variant* operand) RTTR_NOEXCEPT;
Additionally:

added non-const version of variant::get_value()
fixed copy-right year in root CMakeLists.txt file

Fixes #108

Coverage increased (+0.03%) to 92.382% when pulling 55ffd5392ff952011b6d01544dd089f99fc8b47d on acki-m:feature-add-variant-cast into 621cd1df0c9b2d6133aabcf5d3cc129f17a87fac on rttrorg:master.

@gabyx
What do you think?


image : modify http cache-control split

As ToDo message indicates, http cache-control arguments are separated by comma and whitespace. Please give any feedback, thanks!

Do we have a functional test about the http cache?

Do we have a functional test about the http cache?

Our test server doesn't support cache, so nope.
We have a unit test but we only add one value to Cache-Control, so we can't tell the difference between separating by spaces and by commas.
The HTTP/1.1 spec is clear about it being a comma-delimited list (syntax).

separated by a single comma (",") and optional whitespace (OWS)

So instead of Split on ", ", rkt should Split on "," and then Trim.

So I used TrimSpace and then Split on ", " ,  check example here

So I used TrimSpace and then Split on ", " , check example here, in which parts1 case seems better fit.
Thanks @iaguis @alban

This doesn't work if there's no space between the values (which is allowed by the spec): https://play.golang.org/p/0394ds0b_u
I agree with @alban, you need to Split on "," and then TrimSpace each part.

@iaguis Thanks for comments, just modified to Split first, and then TrimSpace.  Please check.

LGTM


No such file "model_params.json"

When I run the train.py, it occurs the error as follows:
FileNotFoundError: [Errno 2] No such file or directory: 'model_params.json'

Hi,
Sorry for the late reply. It needs the parameter file which was missing in the repo. Have added it now. This error should be solved.


Question about extracting ground truth labels

Hello,
I am building a NN for segmentation tasks which only takes point cloud as input but I am stuck in constructing loss function because I cant't extract ground truth labels in Perception dataset. How can we extract ground truth labels (semantic class, 7D bounding box ....) from .tfrecord file? Thanks in advance.
best

Solved


Error when trying to use - issue with constructor

I've installed the package in a Laravel 8 project. I'm trying to run the basic example, code below
use Vemcogroup\SmsApi\SmsApi;

public function test()
{
  $to = '+3248684xxxx'; // commenting out last numbers for privacy
  $from = '+3249920yyyy';  // commenting out last numbers for privacy
  $message = 'This is a test message';

  SmsApi::send($to, $from, $message);
}

And this results in the following error:
Too few arguments to function Smsapi\Client\SmsapiHttpClient::__construct(), 0 passed in C:\project_path\\vendor\vemcogroup\laravel-smsapi\src\SmsApi.php on line 21 and exactly 3 expected

When I look at the code of the constructor in SmsapiHttpClient, it seems to me that you are trying to resolve the 3 parameters from the container, rather than that they need to be passed as parameters. But somehow this doesn't work.
Do I need to register somehow the ClientInterface,RequestFactoryInterface and/or StreamFactoryInterface in my project? If so, please could you explain where and how?

@ndeblauw Hey Nico, didn't get a notification for this issue request.
So yea you are right, after they upgraded to v3 it seems like they changed their constructor and we didn't take account into that as we used their package in a different way.. To get you going this is what we are doing atm, we'll take a look at fixing the package once back from vacation so that it does something similar as to what we are doing with these 3 lines instead:
            $sms = SendSmsBag::withMessage($this->to, $this->text);
            $sms->from = $this->from;
            $sms->normalize = true;
            $service = (new SmsapiHttpClient())->smsapiComService(config('smsapi.token'));
            $sms = $service->smsFeature()->sendSms($sms);

Could you check what version you are using, I have a hunch you are on v1 of the package?

Hi @danijelk , thanks for coming back on this issue! When I couldn't get it to work, I actually wrote my own version of your SmsApi class, with only one difference: I included the curl httpclient and this was enough to get it working... See below.

Indeed I saw it in our script as well now, didn't think of looking at that, update released thanks!


ObsGridItem

This component should display:

[ ] Taxon photo as the background, at a width/height of 172px and border radius of 15px

Overlaying the taxon photo, this component should display:

[ ] Top Left: sound icon if the observation has an associated sound
[ ] Top Right: icon displaying the number of photos in the observation
[ ] Bottom Left: either the ObsStatus component (#434) or the UploadIcon component (#433) if observation needs to be uploaded to the server
[ ] Bottom Left: TaxonName component (#435), with color set to white, only the Common Name showing, a maximum of three lines of text, and a maximum text width of 145px. There should be 5px of space above this component (between it and UploadIcon or ObsStatus)

The color for all elements overlaying the taxon photo should be:

[ ] #FFFFFF (white)

i can take a stab at this

Is there an example w/o a taxon image?

@Chrischuck, this component should never show a taxon image, but it should show an observation photo. Without a photo it should just show a gradient like this:
background: linear-gradient(180deg, rgba(0, 0, 0, 0) 0%, rgba(0, 0, 0, 0.5) 100%), #454545;
background-blend-mode: multiply, normal;
border-radius: 15px;


Support downloading binaries from docker images

needs: https://github.com/docker/actions-toolkit/pull/438

This patch extends the version input parameter to also accept an CSV-style options allowing to specify a different installation source of the docker binaries:

archive (old and still default) - fetches static binaries from download.docker.com

type - must be archive
version - a Docker version to install (example: v27.2.1) or latest (last stable release)
channel - a channel to install from (test or stable)

image (new) - fetches binaries from the moby/moby-bin and dockereng/cli-bin repositories

type - must be image
tag - a tag of the image, can either use a tagged version like 27.2.1 or a branch name like master.

Some examples:
  version: latest                                      # install last stable released version
  version: 27.2.1                                      # install v27.2.1 from download.docker.com
  version: type=archive                          # same as above
  version: version=27.2.1                        # same as above
  version: type=archive,version=27.2.1  # same as above

  version: type=archive,version=27.3.0-rc.1,channel=test # install v27.3.0-rc.1 from test channel

  version: type=image                             # install last stable released version from bin images
  version: type=image,tag=latest           # same as above

  version: type=image,tag=master            # install a cutting-edge version from the `master` branch
  version: type=image,tag=27.2.1              # install v27.2.1

@vvoland https://github.com/crazy-max/ghaction-setup-docker/pull/122 just got merged, you can rebase

@vvoland Added extra commit to test type=image,tag=latest in GHA and seems we don't use the actual version neither when downloading systemd units: https://github.com/crazy-max/ghaction-setup-docker/actions/runs/11582762974/job/32246484640?pr=106#step:5:2171
[  298.001262] cloud-init[2118]: + wget https://raw.githubusercontent.com/moby/moby/latest/contrib/init/systemd/docker.service https://raw.githubusercontent.com/moby/moby/vlatest/contrib/init/systemd/docker.service -O /etc/systemd/system/docker.service
[  298.040071] cloud-init[2118]: --2024-10-29 21:52:40--  https://raw.githubusercontent.com/moby/moby/latest/contrib/init/systemd/docker.service
[  299.046101] cloud-init[2118]: Resolving raw.githubusercontent.com (raw.githubusercontent.com)... <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, ...
[  299.082724] cloud-init[2118]: Connecting to raw.githubusercontent.com (raw.githubusercontent.com)|<IP_ADDRESS>|:443... connected.
[  299.202189] cloud-init[2118]: HTTP request sent, awaiting response... 404 Not Found
[  299.216292] cloud-init[2118]: 2024-10-29 21:52:41 ERROR 404: Not Found.
[  299.226332] cloud-init[2118]: --2024-10-29 21:52:41--  https://raw.githubusercontent.com/moby/moby/vlatest/contrib/init/systemd/docker.service
[  299.244670] cloud-init[2118]: Reusing existing connection to raw.githubusercontent.com:443.
[  299.313128] cloud-init[2118]: HTTP request sent, awaiting response... 404 Not Found
[  299.328273] cloud-init[2118]: 2024-10-29 21:52:41 ERROR 404: Not Found.
[  299.345236] cloud-init[2118]: + true
[  299.354731] cloud-init[2118]: + wget https://raw.githubusercontent.com/moby/moby/latest/contrib/init/systemd/docker.socket https://raw.githubusercontent.com/moby/moby/vlatest/contrib/init/systemd/docker.socket -O /etc/systemd/system/docker.socket
[  299.386189] cloud-init[2118]: --2024-10-29 21:52:41--  https://raw.githubusercontent.com/moby/moby/latest/contrib/init/systemd/docker.socket
[  300.347605] cloud-init[2118]: Resolving raw.githubusercontent.com (raw.githubusercontent.com)... <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, ...
[  300.373431] cloud-init[2118]: Connecting to raw.githubusercontent.com (raw.githubusercontent.com)|<IP_ADDRESS>|:443... connected.
[  300.471471] cloud-init[2118]: HTTP request sent, awaiting response... 404 Not Found
[  300.487020] cloud-init[2118]: 2024-10-29 21:52:42 ERROR 404: Not Found.
[  300.500385] cloud-init[2118]: --2024-10-29 21:52:42--  https://raw.githubusercontent.com/moby/moby/vlatest/contrib/init/systemd/docker.socket
[  300.520196] cloud-init[2118]: Reusing existing connection to raw.githubusercontent.com:443.
[  300.576193] cloud-init[2118]: HTTP request sent, awaiting response... 404 Not Found
[  300.600016] cloud-init[2118]: 2024-10-29 21:52:42 ERROR 404: Not Found.

Added extra commit to test type=image,tag=latest in GHA and seems we don't use the actual version neither when downloading systemd units: https://github.com/crazy-max/ghaction-setup-docker/actions/runs/11582762974/job/32246484640?pr=106#step:5:2171

I guess we could use the commit sha from org.opencontainers.image.revision label or annotation: https://github.com/moby/moby/actions/runs/11584834496/job/32252730725#step:3:59

Does not seem to work on macOS: https://github.com/crazy-max/ghaction-setup-docker/actions/runs/11588305245/job/32265459115?pr=106#step:5:11

  Downloading docker cli from dockereng/cli-bin:26.1.5
Error: Cannot find manifest for darwin/amd64/

@vvoland I'm taking a look to switch to undock but if you want to fix this issue, feel free to open a PR on actions-toolkit

The issue here is that 26.1.5 didn't had the darwin cli builds yet.

@crazy-max  - Opened a PR to get the git revision from org.opencontainers.image.revision:
https://github.com/docker/actions-toolkit/pull/480


Creating new Service Principal to rotate credentials permission propagation issues

The create new service principal section leads to issues not documented here. I performed the steps as described to create a secondary service principal to rotate between. When I applied the new service principal it did not have the correct permissions on resources created in the MC resource group. Specifically, it was unable to access the load balancer created as part of the NGINX  ingress controller.  This caused all ingress traffic to fail.  All of the documentation advises not touching that managed resource group so just granting the needed permissions seems incorrect.
https://docs.microsoft.com/en-us/azure/aks/update-credentials#create-a-new-service-principal

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

ID: 0b842222-f6ac-5266-227f-7e400610e07b
Version Independent ID: 429cd3bf-2691-f633-2e0d-e70e5d8d0e03
Content: Reset the credentials for a cluster - Azure Kubernetes Service
Content Source: articles/aks/update-credentials.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

Thank you @dwolf4 for the feedback.
I was able to successfully follow the article on an existing cluster and confirmed that ingress still works properly.
Closing for now but please follow up if you're still running into the same issue.
#please-close


bandwhich: 0.22.2 -> 0.22.2-unstable-2024-08-13 (fix build)

Diff: https://github.com/imsnif/bandwhich/compare/adfadfa...894ea319838cbfd75b00f08518bca41a9c8e060c
Changelog: https://github.com/imsnif/bandwhich/blob/894ea319838cbfd75b00f08518bca41a9c8e060c/CHANGELOG.md
related to https://github.com/NixOS/nixpkgs/issues/332957
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Wasn't this already fixed by https://github.com/NixOS/nixpkgs/pull/334824? At least postPatch should have been removed.

I didnt saw that pull request.

https://github.com/NixOS/nixpkgs/pull/335092

I didnt saw that pull request.

It was linked to in https://github.com/NixOS/nixpkgs/issues/332957 next to the bandwhich list item.


Both v-model and :list are modified on drag&drop

Hi!
I don't know how to solve the problem I have with Drag&Drop.
Up to now I've had two buttons move up and move down to change the position of the elements of the list. Each change was requesting an update on the server and with a success the objects were moved (hence, every move required an ok from the server, then the list was updated). Being the list stored into the state part of a Pinia store, once they updated Vue proceeded to redraw the UI.
But, if I use the Drag&Drop, I can't have the same behaviour.
I manage the onEnd event, so I proceed with the call on the server once the user has stopped to drag the object. I've supposed that if use the v-model, as written on the documentation, no changes are performed on the model as long I return a false in the function. But it's not the case.
With both v-model and :list the content is changed. That means that I can't use the same function used in moveUp/moveDown, because otherwise I perform another change. The only way to obtain the original status is to add whole new set of functions that are used only for Drag&Drop.
Is this normal? I mean, is normal that both v-model and :list are not immutable?

@LukMas v-model is by-directional but list is not


fix: wrap extend call expression in statement so it's closed with a semicolon

This solves the Module parse failed: Unexpected token error when building a minified bundle. The call expression is inserted as-is, and expressions don't get a semicolon by design.
Closes #8

Hello, is this on the npm already?  I'm still having this error using the latest version.

Hello, is this on the npm already?  I'm still having this error using the latest version.

My bad, I forgot to ask @drcmda to release a new version on npm. Will do that now!

@rnike these changes should be published now as version: 0.1.1. Let me know if that fixes this error for you

@umar-ahmed no more errors after upgrading the package, thank you!


Microsoft.Build.* is not compatible with netstandard1.6

I started off targeting netstandard1.6 basing my assumptions off a template I was using as reference. However, soon discovered these errors.
Severity	Code	Description	Project	File	Line	Suppression State
Error	NU1202	Package Microsoft.Build.Framework 15.7.179 is not compatible with netstandard1.6 (.NETStandard,Version=v1.6). Package Microsoft.Build.Framework 15.7.179 supports:
  - net46 (.NETFramework,Version=v4.6)
  - netstandard2.0 (.NETStandard,Version=v2.0)	BumpAssemblyVersions	G:\Source\Kingdom Software\BumpAssemblyVersions\Prototype\src\BumpAssemblyVersions\BumpAssemblyVersions.csproj	1
Error	NU1202	Package Microsoft.Build.Utilities.Core 15.7.179 is not compatible with netstandard1.6 (.NETStandard,Version=v1.6). Package Microsoft.Build.Utilities.Core 15.7.179 supports:
  - net46 (.NETFramework,Version=v4.6)
  - netstandard2.0 (.NETStandard,Version=v2.0)	BumpAssemblyVersions	G:\Source\Kingdom Software\BumpAssemblyVersions\Prototype\src\BumpAssemblyVersions\BumpAssemblyVersions.csproj	1

I changed the TargetFramework to netstandard2.0 and am still receiving this error. Even after a Rebuild.
My package references are thus:
<ItemGroup>
  <PackageReference Include="Microsoft.Build.Framework" Version="15.7.179" />
  <PackageReference Include="Microsoft.Build.Utilities.Core" Version="15.7.179" />
  <PackageReference Include="System.Xml.XDocument" Version="4.3" />
  <PackageReference Include="System.Xml.XPath.XDocument" Version="4.3" />
  <PackageReference Include="Newtonsoft.Json" Version="11" />
  <PackageReference Update="@(PackageReference)" PrivateAssets="All" />
</ItemGroup>

At the moment, I am not multi-targeting at all. Do I need to account for the 4.6 path in the targeting set?
Thanks!

Have you done a new restore after updating your TargetFramework?

Oh, I think I see... I needed to be more specific in my Version request...

@rainersigwald However, I would suggest, there may be a stale message netstandard1.6 even though the package indeed multi-targets netstandard2.0 and net46. That could mislead some folks into chasing either (or both) of those rabbit holes to no avail.


Can't continue quests when I need to bring item with custom name and lore

What is your Quests version / build number (do not say "latest")?

Describe the problem and how to replicate it.
I have a quest with 3 stanges. First and second stages about finfing locations and in third I need to bring some items to NPC. When I complete second stage nothing happen. Even if u use /quest command I just see an error in chat and console.

error on second stage complete

error on questsadmin reset command

error on questadmin nextage command

Any console errors to share?
In description

If applicable, can you provide an example quest from quests.yml?
custom27:
    name: name
    ask-message: text
    npc-giver-uuid: db08d0df-1083-497e-a243-defdc26269a8
    stages:
      ordered:
        '1':
          locations-to-reach:
          - world 299.0 59.0 -18.0
          reach-location-radii:
          - 5
          reach-location-names:
          - Глубь пещеры
          complete-message: text
        '2':
          locations-to-reach:
          - world 282.0 57.0 -20.0
          reach-location-radii:
          - 5
          reach-location-names:
          - Пещера
          complete-message: text
        '3':
          items-to-deliver:
          - ==: org.bukkit.inventory.ItemStack
            v: 3953
            type: REINFORCED_DEEPSLATE
            amount: 2
            meta:
              ==: ItemMeta
              meta-type: UNSPECIFIC
              display-name: '{"text":"","extra":["Часть древних ворот"]}'
              lore:
              - '{"text":"","extra":["Часть древних ворот, разрушеннных несколько
                веков назад"]}'
          npc-delivery-uuids:
          - db08d0df-1083-497e-a243-defdc26269a8
          delivery-messages:
          - Мне необходимо 2 части, а тут только одна
    rewards:
      items:
      - ==: org.bukkit.inventory.ItemStack
        v: 3953
        type: NETHERITE_HELMET
        meta:
          ==: ItemMeta
          meta-type: ARMOR
          display-name: '{"text":"","extra":[{"text":"Незеритовый шлем от незнакомца","obfuscated":false,"italic":false,"underlined":false,"strikethrough":false,"color":"light_purple","bold":false}]}'
          lore:
          - '{"text":"","extra":[{"text":"Награда за прохождение квеста","obfuscated":false,"italic":false,"underlined":false,"strikethrough":false,"color":"light_purple","bold":false}]}'
          - '{"text":"","extra":[{"text":"Поиск сквозь века","obfuscated":false,"italic":false,"underlined":false,"strikethrough":false,"color":"light_purple","bold":false}]}'
          enchants:
            minecraft:respiration: 10
          repair-cost: 7
      quest-points: 1
      commands:
      - bc Игрок <player> прошел квест "Опасное местечко" от незнакомца из пещеры
      details-override:
      - '&dНезеритовый шлем от незнакомца'
      - '&f1 &aQкоин'
    planner:
      cooldown: 1209600
      override: false
    options:
      allow-commands: true
      allow-quitting: true
      ignore-silk-touch: true
      use-parties-plugin: true
      share-progress-level: 1
      same-quest-only: true
      share-distance: 0.0
      handle-offline-players: false
      ignore-block-replace: true

Thanks for reporting this issue. Would you mind trying this dev build, and reply with any new errors?
https://ci.codemc.io/job/PikaMug/job/Quests/487/

@PikaMug The issue is still here. Nothing happen when complete second stage. Manually get third stage impossible too
Error: https://mclo.gs/I4mapWq

@StKillReal That error is from build 486. Please reply with the  new error from build 487.

@StKillReal Thank you. I went to test this just now and noticed that your 'custom27' quest doesn't actually load because it's missing a finish-message. It's possible that the problem lies with a different quest. I'll try to replicate, but please fix 'custom27' and let me know if the issue persists or you get any new errors afterward.

@PikaMug doesn't work even with finish-message for some reasons

@StKillReal I'm still having trouble replicating this, but have made some changes which hopefully might work on your end. Please give this build a try: https://ci.codemc.io/job/PikaMug/job/Quests/488/

@PikaMug All works fine now, thakns!


API  the xmtp_mls client needs to be adapted to accept an abstract XmtpApiClient & create a WASM-friendly implementation in xmtp_api_grpc_gateway

I think this is solved from #949


React-dnd hijacks out of context "draggable" elements

When calling useDrop it will effectively remove all dragging capabilities from your app, even if they are outside the DndProvider container.
See:
https://codesandbox.io/s/festive-sky-muxxt?file=/src/index.js
Try dragging the red square, it's marked as draggable which means it should show the HTML5 dragging overlay.
The issue is that I'm allowing components to be dragged in from outside the DndProvider because it's using react-grid-layout, but react-dnd breaks it.

Also seeing this. react-dnd version 14.0.4. We have a table component w/ draggable rows that uses react-dnd, but not every other component in the system has been updated to use it as well, and it feels like if its usage is isolated to one place it shouldn't be taking over all drag events

+1
Is there a solution now?

Reading the documentation I found one way to approach this use case.
Works for me!
// ...
const DnDRef = React.useRef(null);
// ...
<div
      key={`${headerGroup.id}-${headerGroupIndex}`}
      className={`${c("tr header-group")}`}
      ref={DnDRef} <-- Create your context
>
  <DndProvider
        key={`${headerGroup.id}-${headerGroupIndex}-dnd-provider`}
        backend={HTML5Backend}
        context={DnDRef} <-- Use the context you create
    >
    {// ...}
  </DndProvider>

It seems that there is something wrong with RafaelGB's solution. DnD don't work correctly.
https://codesandbox.io/s/infallible-bardeen-8fh98f?file=/src/index.js

My use case was to fix an incompatibility with an architecture element used by my component (a plugin). I was able to fix it but finally dispensing with react-dnd.
The problem came in the usedrop declaration since it affects all those components with the "onDrop" event loaded disabling them.
Depends of your case just use native dnd

Thank you for your reply which solved my doubt!
Some of my imported components use native dnd.  These components and react-dnd are not compatible at the same time.😭

Thank you for your reply which solved my doubt!
Some of my imported components use native dnd. These components and react-dnd are not compatible at the same time.😭

I found the reason why dnd don't work.(referring to #3344)
function DndProviderWrapper({ className, children }) {
  const context = useRef(null);
  const dndArea = context.current;
  const html5Options = useMemo(() => ({ rootElement: dndArea }), [dndArea]);
  return (

      {// it is important that DndProvider don't render if dndArea is null }
      {dndArea && (
        <DndProvider backend={HTML5Backend} options={html5Options}>
          {children}
        </DndProvider>
      )}

  );
}

// ...

  <DndProviderWrapper>
    {// ... your component used react-dnd}
  </DndProviderWrapper>
 {// ... your component used native dnd}

Thank you all for addressing this !
I had the same issue this week and it wasn't clear what exactly was causing the problem until I figured out it was the DndProvider + useDrop combination.
Ytime's solution works great for my use case (thanks for the solution!), although I had to tweak it a little :
const DndProviderWrapper: React.FC<any> = ({ className, children }) => {
    const context = useRef(null);
    //Make the dndAre a part of the component state otherwise, some times, the content doesn't renders
    const [dndArea, setDnDArea] = useState(context.current);
    //additionally, add a useEffect to track the context reference => might be overkill performance wise but works for my use case
    useEffect(() => {
        setDnDArea(context?.current);
    }, [context])
    const html5Options = useMemo(() => ({ rootElement: dndArea }), [dndArea]);
    return (

            {/* it is important that DndProvider don't render if dndArea is null */}
            {dndArea && (
                <DndProvider backend={HTML5Backend} options={html5Options}>
                    {children}
                </DndProvider>
            )}

    );
}

I had the exact same problem and the above fixed my issue. Should consider merging this into the next release.


Bump google-cloud-bigquery version to 1.3.0

This is currently at 0.29.0. The latest is 1.1.0, and 0.29.0 doesn't support some features that are required with recent versions of pandas-gbq

Thanks @maxim-lian -- those Google engineers... they work quick :). Looks like the latest release is 1.3.0
Summary of changes from 0.29.0 to <IP_ADDRESS>:
0.30.0:

Add delete_contents to delete_dataset

0.32.0:

Use job.configuration resource for XXXJobConfig classes

1.0.0

Remove deprecated Client methods

1.2.0:

Add column based time partitioning

1.3.0:

Add client.load_table_from_dataframe()

...
I think the job.configuration change in 0.32.0 might require a code change, but otherwise, I think there are a lot of nice new features here that we can take advantage of. We'll definitely to check out which methods were removed in 1.0.0 as well.

Thanks @drewbanin !
A few other dependencies that are out of date (a couple really out of date):
dbt 0.10.1 has requirement celery==3.1.23, but you'll have celery 4.2.0 which is incompatible.
dbt 0.10.1 has requirement google-cloud-bigquery==0.29.0, but you'll have google-cloud-bigquery 0.30.0 which is incompatible.
dbt 0.10.1 has requirement networkx==1.11, but you'll have networkx 2.1 which is incompatible.
dbt 0.10.1 has requirement pytz==2017.2, but you'll have pytz 2018.4 which is incompatible.

+1 for fixing the pytz and other requirements. In particular, some Singer taps don't support pytz=2017.2.

should we use client.load_table_from_dataframe() for seed?


Optimize Socrata (SODA) API usage

Hi there - I stumbled upon your usage of Miami Dade's Socrata portal and think this project looks great. I did notice that your API calls aren't using the newer, higher performance versions of the APIs.
For example,  https://opendata.miamidade.gov/resource/kw55-e2dj.json can be changed to https://opendata.miamidade.gov/resource/vvjq-pfmc.json. In addition to being faster overall, these new API endpoints include new features like more feature parity for SoQL with SQL such as LIKE and IN queries and much more.
You can learn more about the new features and how to find the new API endpoints at http://dev.socrata.com/changelog/2015/04/27/new-higher-performance-apis.html
Hope this helps!

:+1:
Let me know if there's anything I can do to help!

@marks, had no idea. We'll put this in on our next push. Thanks for the heads up!

@ErnieAtLYD sweet! You guys rock.


Vs Other Similar Languages

Wondering how you stack against:

https://github.com/carp-lang/Carp
https://github.com/LuxLang/lux
http://typedclojure.org/
Typed Racket

What are the advantages and disadvantages?

None of those are pure, Haskell and Hackett are.

Two big things:

Purity and type system features, as @joehillen mentions. TR and core.typed have radically different type systems that are designed to support gradual typing, and they attempt to support idiomatic Racket and Clojure, respectively. That comes with a whole host of pros and cons, but one of the biggest missing features is typeclasses, which none of those languages support.

Hackett is also designed to support much more type-level metaprogramming than most other languages, including Haskell. Lux may compare somewhat, due to the way it exposes the compiler to macros, but it doesn’t support hygienic macros, which are pretty important in my experience for being able to write expressive macros.


[#26126] Disable Debezium xlang test against Samza runner.

Continue to get some signal from Samza by filtering Debezium from running against it. #26126 to track this eventually being resolved so we can re-enable it.
This PR to be cherry picked to resolve release blocker: #26241.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run XVR_Samza PostCommit

R: @jrmccluskey
I'll send a cherry pick after this is merged.


[5.3] Write output if a job is re-queued

When the job 'fails', it only outputs the Failed: message if you have the --tries specified, and only if it reaches that amount.
I propose to add a message to indicate that a job has encountered an exception,
and has been re-queued, to let the developer know something is up.

You don't know the job will be "re-queued" anytime there is a job exception.

afaik it always re-queues it, it only checks if it fails at the start.


chore(ci): add manual workflow for release

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows Conventional Commits
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Refactoring
[ ] Code style update
[x] Build or CI related changes
[ ] Documentation content changes

Closes #1242
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Please, rename PR to ci: add manual workflow for release


Slimjet

Please also Add for Slimjet "Recently visited sites" and "most frequently visited sites" to History AutoDelete

From the Slimjet website:

Since Slimjet is based on the Chromium project, it remains compatible with almost all of the extensions and plugins designed for Google Chrome. You can download extensions and plugins for Slimjet from the Chrome web store.

Install it from https://chrome.google.com/webstore/detail/history-autodelete/bhfakmaiadhflpjloimlagikhodjiefj . If you try a Chromium extension and it doesn't work, contact Slimjet instead of creating issues on Github repositories.
Anyway, this specific extension isn't being worked on anymore.  You can use Firefox with Historia instead.


service: add coco feature in Cargo.toml

Add feature coco to Cargo.toml, so that confidential containers can apply this feature to use nydus to download images.

We have fixed a go version, which fixes the smoke test failure. Could you please help to force push again?


Functions become plain object when they're moved

When I'd like to clone an object to a brand new one and there's a function in the source objects, the function becomes a plain object in the final result.
it works fine:
var a = {
  foo: function bar() {}
};
var result = deepAssign(a, {});
console.log(typeof result.foo); // --> 'function'

but in this case the function becomes a plain object:
var a = {
  foo: function bar() {}
};
var result = deepAssign({}, a, {});
console.log(typeof result.foo); // --> 'object' :((

@ruffle1986: this is intended: properties of functions are not enumerable:
for(prop in a.foo) {
  console.log(prop);
}
//=> no output

You may also compare this to nodes Object.assign implementation:
function bar() {}

Object.assign(bar, {}) //=> [Function: bar]
Object.assign({}, bar) //=> {}

See ECMAScript® 2015 Language Specification for details about Object.assign.

 And if you check how Object.assign works (for example in Chrome 45), you'll see that when a function is in an object to be assigned, the function in it remains to be a function after the operation and that's what I want.
var a = { foo: function bar() {} };

(typeof Object.assign(a, {}).foo); // => "function"

(typeof Object.assign({}, a, {}).foo); // => "function"

Sorry but I can't derive a valid behavior for deep-assign in a canonical way from your last example, simply because Object.assign is not recursive.
At least I think your solution introduces some unexpected/inconsistent behavior:
function foo() { return 'foo-return'; }
function bar() { return 'bar-return'; }

// PR #3
var deepAssign = require('deep-assign');
var result =  deepAssign({fn: foo}, {fn: bar});
console.log(result.fn.name);   // => 'bar'
console.log(result.fn());      // => 'bar-return'

// but manually Object.assign
var result =  { fn: Object.assign(foo, bar) };
console.log(result.fn.name);   // => 'foo'
console.log(result.fn());      // => 'foo-return'

@schnittstabil I don't want to merge the functions themselves. I'd like to merge objects which hold functions and I want deep-assign to not transform them into plain object.
Current (wrong) behaviour:
var sourceA = {};
var sourceB = {
  foo: function () {
    return 'Hurray!';
  }
};
var target = {};

deepAssign(target, sourceA, sourceB);

// and then I'd like to use foo() but I can't

target.foo() // => Uncaught TypeError: foo is not a function

Expected:
var sourceA = {};
var sourceB = {
  foo: function bar() {
    return 'Hurray!';
  }
};
var target = {};

deepAssign(target, sourceA, sourceB);

// I can invoke foo because it's still a function and actually this is how Object.assign works.

target.foo() // => "Hurray!"

@ruffle1986 I am aware what you expecting, but I am also aware of the readme.md:

Recursively assigns [like in Object.assign] own enumerable properties of source objects to the target object and returns the target object. Additional source objects will overwrite previous ones.

And I am also aware of Javascript:

In JavaScript every function is actually a Function object. [from MDN]

The problem is, that I don't see how this PR satisfies these requirements, without introducing unexpected behavior.
E.g.:
function foo() {
  return 'foo';
}

foo.async = function(cb) {
  cb(null, 'foo.async');
}

function bar() {
  return 'bar';
}

console.log(Object.assign(foo, bar).async);           //=> [Function]

// PR #3: base case
console.log(deepAssign(foo, bar).async);              //=> [Function] LGTM

// PR #3: one step deeper
console.log(deepAssign({fn:foo}, {fn:bar}).fn.async); // => undefined  WAT?

I am absolutely willing to support your request, but I think the solution is not quite so simple.

@sindresorhus: me and @schnittstabil are done. You can merge it if you find it good.

Perfect. Thanks for working on this @ruffle1986 and @schnittstabil :beers:

I did a major release as this slightly changes behaviour and just to be extra careful.

sure. cool. not at all. :+1:


 is not replaced when using v-bind={is: ...}

Version
3.0.0
Reproduction link
https://codesandbox.io/s/kind-fire-2p58g?file=/src/App.vue
Steps to reproduce
Open the Link, see the underlined error / Its not running
What is expected?
To be able to bind is with the v-bind="obj" syntax
What is actually happening?
It only works with a separate is attribute

Replacement can only work reliably if the presence of the is attribute is guaranteed. That is not the case when you are using the define object syntax, as that by nature means that there is in dynamic JavaScript expression involved, which during runtime, may or may not return an object with an is property.
While we could make the compiler create the necessary dynamic component resolution code whether or not this risk exists, I don't think it makes sense to support this kind of foot gun.
Right now, the compiler expects the presence of that is attribute in order to successfully transform this piece of tempered into a dynamic component resolution. What we could do, is to Warren in the console when the is property is not statically defined.

This problem lost me an entire afternoon. I was trying the same in the vue 2 codepen for the same feature and it was working, so it ended up being a really frustrating day.
It might help someone who will have the same problem in the future:
Because I was iterating an array (json schema to generate inputs) and calling a method on its elements to determine the component name and its props, I really needed this feature, because I was not going to process the same algorithm for every property needed for the component.
In the end I created a new component with a sole  tag with an explicit "is" and the rest of the properties separated. I then use a computed prop to run my algorithm passing the prop (json schema object) as the algorithm starting point.

fixed this problem this way:
    <component
        v-bind="(({ is, ...o }) => o)(componentOptions)"
        :is="componentOptions.is"
    </component>

yes, it looks hacky but it works and this way is won't show up in the rendered HTML as an attribute on the rendered component.

I think you ate killing some performance there because that function will lead to unnecessary rerendering. Why not just v-bind="{...obj, is: undefined}"?

that will still render on the html is="undefined"

@kepi0809 you sure? Afaik, undefined props are skipped

@kepi0809 you sure? Afaik, undefined props are skipped

weird... this was the first approach I tried to fix this problem and it didn't seem to work then, but now I tested it again and it works with both is: null and is: undefined.
Thank you for for pointing again in that direction :+1:

This problem lost me an entire afternoon. I was trying the same in the vue 2 codepen for the same feature and it was working, so it ended up being a really frustrating day.
It might help someone who will have the same problem in the future:
Because I was iterating an array (json schema to generate inputs) and calling a method on its elements to determine the component name and its props, I really needed this feature, because I was not going to process the same algorithm for every property needed for the component.
In the end I created a new component with a sole tag with an explicit "is" and the rest of the properties separated. I then use a computed prop to run my algorithm passing the prop (json schema object) as the algorithm starting point.

met the same problem, and is there any solution, my friend? try several ways and only fixed the props as a computed data could meet the demand.
<template>
  <KeepAlive>
    <!-- it works -->
    <Component :is="currentTab.component" v-bind="props" />
    <!-- it does not work -->
    <Component :is="currentTab.component" v-bind="currentTab.props" />
  </KeepAlive>
</template>

<script lang="ts">
import { Vue, Component } from "vue-property-decorator";
import { ITabType } from<EMAIL_ADDRESS>import { Channel, Complay } from "@/UnionTask";

enum ETab {
  channel,
  complay,
}

interface ITabConfig extends ITabType<ETab> {
  component?: typeof Vue;
  props?: Record<string, any>;
}

@Component({
  components: {
    Channel,
    Complay,
  },
})
export default class ChannelUnion extends Vue {
  unionInfo: Record<string, string> = {}; // union info

  channelListInUnion: Record<string, string>[] = []; // channel list

  currentTab: ITabConfig = this.tabConfig?.[0];

  /**
   * @description
   * Dynamic parameters of dynamic components
   * Because tabConfig is an array props cannot be declared directly in the object,
   * an additional dynamic props must be maintained independently
   * @see https://github.com/vuejs/core/issues/2279#issuecomment-800608949
   */
  get props() {
    if (this.currentTab.value === ETab.channel) {
      return {
        unionInfo: this.unionInfo,
        channelListInUnion: this.channelListInUnion,
      };
    }
    return {};
  }

  get tabConfig(): ITabConfig[] {
    return [
      {
        value: ETab.channel,
        label: "channel task",
        component: Channel,
        props: {
          unionInfo: this.unionInfo,
          channelListInUnion: this.channelListInUnion,
        },
      },
      {
        value: ETab.complay,
        label: "complay task",
        component: Channel,
      },
    ];
  }
}
</script>

<style lang="scss" scoped></style>


[send-and-read-emails] return only five threads

Description

Returns only 5 threads, instead of 50, for send-and-read-emails use case

Added key prop to DisplayEmail component to remove the warning Each child in a list should have a unique "key" prop. in App.js for ReadEmails

License
I confirm that this contribution is made under the terms of the MIT license and that I have the authority necessary to make this contribution on behalf of its copyright owner.

@krisgardiner Doesn't look like there are any issues with code highlighting in App.js because the line change is after all the highlighted code. Marked this as will not do https://nylas.atlassian.net/browse/TEM-162


renamed groupByRelationKey to groupByKey

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

Thanks!


Issue with injected Svelte styles

Not 100% grounded and completely investigated, but I have noticed that on some pages, content elements are hidden whenever this Chrome Extension is activated.
A short investigation led to the following assumption:
The extension/DevTools uses Svelte and injects its styles at runtime. This includes a generic hidden class. Because it is injected in the head as a style block, the specificity is higher than the same class referenced before.
Please see the following screenshots:
Without extension:

Activated extension:

This is the injection:

If this is correct, it will disrupt functionality on many websites, as this is also a default class of Tailwind. If possible, it should be scoped.

Hey @svenadlung ! Thanks for the bug report, yeah I've tried to fix this by using class names that ought to be unique but this only gets you so far. I'll definitely try to get the styles scoped somehow. Best way would be a web component. Second best I guess just mangling the class names, last time I checked Svelte didn't provide this out-of-the-box.


Create a README.es.md

Hello! I'd like to contribute with this project by offering a translation of the README file into Spanish so spanish talking people can have a guide about this app. :+1:

Hi @LuigiBaute ,
I'm closing this issue for now. Please submit a PR when you are ready and we will review it.
Thanks.


Gym 0.22.0

update required minimum gym version number
added pygame as a requirement because it is not picked up by the gym requirements
getting rid of CustomBipedalWalkerEnv because the functionality of changing the gravity is covered by CARLEnv (same for CustomLunarLanderEnv)
add high game over penalty for LunarLander by a wrapper

Our basic installation uses classic control right now (and I think that's how we want to keep it) which needs pygame in gym, so I'm not sure using it as an extra makes sense for now. I think they have it as an extra because there it makes a bit more sense to only use mujoco than to only use Brax in CARL.

No, it's great you noticed, I probably wouldn't have looked :D
Anyways, the PR looks good to me and as if the gym refactoring made our code much nicer. Thanks, Caro, I'll just merge.

@eddiebergman thanks for looking over this:) before it was not an extra requirement for classic_control so I didn't bother looking 😅
for now, we would leave it like Theresa said (until we later wonder why we have pygame in our requirements:D)


Mypy and tests are failing for new RC providers prepared on 28th September, 2022

We've automated creating a PR upon changing the setup.cfg with make test-rc-deps target. However we're observing few mypy failures and tests failure that can be seen in the PR: https://github.com/astronomer/astronomer-providers/pull/673
We need to fix the failures observed in the checks for the above PR.

this is same as https://app.zenhub.com/workspaces/airflow-providers-team-61a89ee7fbf048001dc8378d/issues/astronomer/astronomer-providers/702


RDMO 2.1.1

coverage: 91.475%. remained the same
when pulling 263fa92082357585df6f70e923388b2d13c35979 on dev-2.1.1
into 0b6ccec4038d2dcd3180531d0e86894b6774c99a on main.


Tagging fails near right of wide dataset

https://proxy.hxlstandard.org/data/cb99ae

Closing -- looks like this is just a problem with excessively-long query strings in the HXL Proxy.


Add ZKFair

This PR adds a upcoming L2 ZKFair (previous StableNet):
ZKFair is the first community ZK-L2 based on Polygon CDK and Celestia DA, championing fairness.
websites: https://zkfair.io/
socialMedia: https://docs.zkfair.io/
documentation: https://twitter.com/ZK_fair/
testnet is up, mainnet up due to Dec 20.

Hi, I have merged the master branch.

Hi, @lucadonnoh. Could you tell me what wrong with the auto merge? It says error happened when run yarn test. But it passes in my local enviroment.

there is no need to merge master everytime, it will be done automatically when put in the merge queue, btw i'll look at the issue

Hi, @lucadonnoh. Would u mind to make a approval? It is quite urgent for us. Thank you very much.


bug: Customize phonegap-plugin-barcodescanner and ionic cordova plugin add cordova-plugin-network-information

Hi Team,
In our project we are using

Customize phonegap-plugin-barcodescanner

and

ionic cordova plugin add cordova-plugin-network-information

We are getting conflict issue between these two library.Once i Scan the Barcode after that i check for the presence of network at that time app gets crashed.I can't find crash log too!! could you please suggest some solution for this problem.

Thanks for the issue. I am going to close this as Ionic Framework does not manage these native plugins. I recommend you post your question on our forum or our slack.


Question: How are min and max of cumulative variables assigned?

I created a routing problem and added some dimension to it. A solution assignment is found and I want to know the cumulative value at each index. I noticed that the CumulVar of an assignment does not only have a Value method but also Min and Max methods. Apparently the cumulative variables are implemented in such a way that they can represent intervals. I can see how setting

slack_max>0
fix_start_cumul_to_zero=False

introduces an ambiguity for the cumulative variables as their is a choice in how to start and how much slack to add at each stop. But
Question: How are the Min and Max at each index computed?
PS: Same question on stackoverflow: https://stackoverflow.com/questions/51371496/how-are-min-and-max-of-cumulative-variables-assigned, still figuring out where to post questions about or-tools

First CumulVar is computed as follow:
j == next(i), cumul(j) = cumul(i) + transit(i) + slack(i)
https://github.com/google/or-tools/blob/master/ortools/constraint_solver/routing.h#L383
without any constraints slack will be bound by zero to Max slack var (defined during the Dimension creation)
Now, for Time Windows, the solver use some propagators to determine the Min/Max at each node according to the previous and following node constraint(s) (i.e. TimeWindows constraints can shrink from  the slackVar=[0,MaxSlackVar] range)
ps: Usually questions should be post on the
<EMAIL_ADDRESS>(this is barely the "only" source other team members regularly look at).
But hey! OSS outside world so SO and github issues is also (de facto) available so I try to look at them too..., note for github in an "unforeseeable" future we would like to keep it only for bug(s) and feature(s) request as an issue tracker should be used (i.e. move questions to SO/GoogleGroup :wink: or close it directly :disappointed:) .

j == next(i), cumul(j) = cumul(i) + transit(i) + slack(i)
Ok, this equation is helpful. So I tried confirming that it holds by printing for every index the cumulative value, the transit value and the slack value via

cumul = assignment.Value(time_dimension.CumulVar(i))
slack = assignment.Value(time_dimension.SlackVar(i))

If I set fix_start_cumul_to_zero = True then I can confirm the equation, sample output:

i: 0
cumul(0): 0
slack(0): 5
transit(0,1): 10
i: 1
cumul(1): 15
slack(1): 0
transit(1,2): 10
i: 2
cumul(2): 25
slack(2): 0
transit(2,3): 10
...

but if I set fix_start_cumul_to_zero = False then I numbers don't add up:

i: 0
cumul(0): 0
slack(0): 0
transit(0,1): 10
i: 1
cumul(1): 15
slack(1): 0
transit(1,2): 10
i: 2
cumul(2): 25
slack(2): 0
transit(2,3): 10
...

Clearly a waiting time of slack(0)=5 is implied here as well. To understand what is actually going on I print the Max and Min of each variable using
cumul_min = assignment.Min(time_dimension.CumulVar(i))
cumul_max = assignment.Max(time_dimension.CumulVar(i))

A sample output is then:

i: 0
cumul(0): [0,70]
slack(0): [0,10]
transit(0,1): 10
i: 1
cumul(1): [15,80]
slack(1): [0,10]
transit(1,2): 10
i: 2
cumul(2): [25,90]
slack(2): [0,10]
transit(2,3): 10
...

But now it becomes trickier to interpret the assignment. cumul(0) lives in the interval [0,70] and slack(0) in the interval [0,10] but clearly there are dependencies between the choice of values, like cumul(0)=70 implies slack(0)=0 meaning I can not start late and have a large waiting time.
It seems to me that this affects also your example in https://developers.google.com/optimization/routing/cvrptw
Because what does "Total time of all routes: 376min" mean when the cumulative variable is in an interval? Shouldn't that imply that there is a fast and a slow solution? One that starts as late as possible with short waiting times and one that starts as early as possible with long waiting times?

Yes to all your remarks, I figure out the same.
Total time of all routes here is the minimum time use for completeness i.e. Min(routing.CumulVar(routing.End(vehicle_N)).


Fixed inner nullability of JsonNullable properties.

just a draft for review; dont merge

Would be awesome to have this integrated soon :-)