Multiprocessing and PyOctree error

Hello,
My code uses PyOctree package and works fine.
I'm now trying to run my code in parallel using Python Multiprocessing.
I get this error and I have no idea about hot to solve it:
File "stringsource", line 2, in pyoctree.pyoctree.pyoctree.PyOctree.__reduce_cython__
TypeError: no default __reduce__ due to non-trivial __cinit__

How could I solve this error?
Thanks,
Cheers,
Davide
PS: I can provide more code if needed

Hi Davide,
I assume this error relates to the PyOctree class? My guess is to add a repr function to this class, similar to what I have in the PyOctnode class.
I will test this theory and get back to you. The question may then be what should this class return.
Cheers,
Michael

Sorry, should be reduce function, not repr

Hi Michael,
thanks for your reply.
So you are suggesting to add a reduce function to the PyOctree class. I can try.
What are the input and the output of this function?
Do I have to call this function somewhere else in the code?
Thanks,
Cheers,
Davide

Hi Davide,
I don't know much about pickling/unpickling of extension modules such as PyOctree, but it appears there are several ways of making the PyOctree classes picklable. A __reduce__ method is just one of these. See here. The __setstate__ and __getstate__ methods are also a higher level option.
I found quite a few links on this topic, some of which include:

https://github.com/pele-python/pele/issues/34
https://stackoverflow.com/questions/36301322/pickle-cython-class-with-c-pointers
https://snorfalorpagus.net/blog/2016/04/16/pickling-cython-classes/

You don't need to call these methods directly. You just need to provide them and they will get called automatically during pickling / unpickling. The __getstate__ methods store the current data in picklable python objects (numpy arrays, lists etc), which are then used to recreate the PyOctree classes when unpickled via __setstate__. All that should be needed to recreate the class are the initial inputs i.e. pointCoords and connectivity. That is:
tree = ot.PyOctree(vertexCoords3D,polyConnectivity)
Therefore, my guess is that the __reduce__ method for the PyOctree class might be something like:
cdef __reduce__(self):
    return (PyOctree,(self.vertexCoords3D,self.polyConnectivity))

However, vertexCoords3D and polyConnectivity are not stored in the class by default. So we will need to either modify the __cinit__ method to store them (which is undesirable, as it will use more memory).
Alternative we could add __setstate__ and __getstate__ methods. I think this would essentially involve converting and storing the values from the self.thisptr.vertexCoords3D and self.thisptr.polyConnectivity within PyOctree class as a numpy array during pickling i.e. using __getstate__. When unpickling, i.e. using __setstate__, these would be returned to recreate the PyOctree object from scratch.
Cheers,
Michael

@DavideBassano @mhogg
I implemented it in https://github.com/mhogg/pyoctree/pull/56


Error

Hey, was trying this out to see if I could use it for something I want to do, but I keep getting these errors when I try logging in!

This is a known thing to happen if you're using Unity with a version higher than 5.4.
Since the lib is a bit outdated, because I'm waiting for some discord features, try to stay below 5.4.

Do you have an estimated time on when you'll be able to update it to 5.4? :) Thanks for quick response!

I'm sorry I don't have an estimated time because the discord devs don't know either when it will be there.

That's fair enough, I'll keep my eye out, Thanks for letting me know! :)

Forgot to close this when you updated the api, so I'll close this now, thank you :)


Optimize overwrite and delete commits.

This uses manifest metadata to avoid filtering manifests that cannot contain deleted files in overwrite and delete operations.

I think it is a substantial feature, can we add a couple of tests?

Do we want to overload deleteFile(DataFile) in StreamingDelete to use this functionality? Right now, it will call deleteFile(CharSequence) under the hood.

Once this is merged, I'll update #351

@aokolnychyi, the existing tests cover correctness. This just speeds up the operations by ignoring manifests that can't match.


repetitions_per_point in webgazer-validate plugin?

@jodeleeuw the webgazer-validate plugin docs page lists a repetitions_per_point parameter, which is "The number of times to repeat the sequence of calibration points". There's no such parameter in the actual plugin. Should I implement the repetitions parameter in the plugin and change "calibration" -> "validation" in the docs parameter description? Or remove this parameter from the docs?

Oops. Totally sloppy on my part!
Let's remove the parameter from the docs. And let's do that on main so that it deploys as part of the v6.3 docs. Thanks!


Cannot type inside the editor

I'm trying to use SimpleMDE as my blog content creation in Laravel but it seems that I can't type inside the editor and either no response from the console.log. Is there anything wrong in my configuration or something? It's hard to find the solution on Google so I decided to ask here. Please take a look at my script:
import 'simplemde/dist/simplemde.min.css';
import SimpleMDE from 'simplemde'

var simplemde = new SimpleMDE({
    element: document.getElementById('markdown-editor'),
    autosave: {
        enabled: true,
        delay: 60000,
        uniqueId: '_' + Math.random().toString(36).substr(2, 9)
    },
    forceSync: true,
    placeholder: "Write article here",
    renderingConfig: {
        codeSyntaxHighlighting: true
    },
    spellChecker: false,
})

simplemde.codemirror.on('change', function () {
    console.log(simplemde.value())
})

simplemde.value("This text will appear in the editor");

Did you resolve this? I'm also on laravel and having the same issue - seems to work when removing vue, but I can't work out how to get them both working at the same time.
(I'm actually using easyMDE, as this project seems dead; but equally had little luck finding many solutions on google)


add impacted methods mojo

I have read the code in this PR and left you some comments. But your PR does not have a description, so I do not know how it fits into the grand scheme of things. Also, I do not understand how the code works, so if you can add some more comments, that would be great. @mmk-1

Old version of method-level RPS. New version in PR #79


Doc/cluster configuration

Description
Documentation about cluster configuration

Closed because there are files that must not be here


Clean up and localize several NPC skill variants

This is about a quarter of all the skill variants to check, so there will be more PRs to do here.
dbs left to do:
menace
monsters of myth
mwangi
gray death
npc gallery
one shot
outlaws
bestiary 1-3 (These should be fine)
dark archive
PFS (all, these will likely need a lot of fixing)
frozen flame (I did part of these)
Strength of Thousands
Slithering
Travel Guide
Troubles

Conflicts!

Fixed


Remove useless stuff in csproj

People will start making plugin, so we should show good example.

Note for LiquidPlus, you need Altcoins version of BTCPayServer. This is done adding Altcoins=true to the project reference.
It seems that using Configuration=Altcoins-Debug doesn't work, there is a build error saying ElementsBTCPayNetwork isn't available.
I don't understand why it wouldn't work, as the project reference assembly indeed contains ElementsBTCPayNetwork. (I checked with ILSpy)

I think it is because that whole btcpay is running under that assembly. The
referenced btcpay is running under the plugin configuration which would be
without altcoins setting if only in debug config
On Fri, Jan 20, 2023, 8:29 AM Nicolas Dorier @.***>
wrote:

Note for LiquidPlus, you need Altcoins version of BTCPayServer. This is
done adding Altcoins=true to the project reference.
It seems that using Configuration=Altcoins-Debug doesn't work, there is a
build error saying ElementsBTCPayNetwork isn't available.
I don't understand why it wouldn't work, as the project reference assembly
indeed contains ElementsBTCPayNetwork. (I checked with ILSpy)
—
Reply to this email directly, view it on GitHub
https://github.com/Kukks/BTCPayServerPlugins/pull/1#issuecomment-1398012590,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAN357RFPYMWLZ7L3WOUJMDWTI5GPANCNFSM6AAAAAAUA7LH7U
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Clicking the fold controls in the gutter opens changes magnification.

VSCode Version: Version 1.18.0-insider

Steps to Reproduce:

Make some changes to any file that has some Source Control. Specifically foldable changes.
Click the fold controls in the gutter.

Reproduces without extensions: Yes

I Would hope that's not the expected behavior. It is wicked annoying.

#36237


Machine Learning Notebook

I have created a machine learning tutorial notebook for tomorrow's portion of the hackathon. The data are hosted on both google cloud and cheyenne and can be downloaded from within the notebook. Please review and let me know if there are any issues.

@djgagne, things here generally seem good, looking forward to this tutorial tomorrow!


Added event reporting to a file for insights

What are we thinking for rotation of this file?

@stevendanna Re: file rotation - I updated the description.

@elliott-davis Thanks for the update.  Overall this looks reasonable to me.  I think that in the long run we are going to want a more sophisticated gen_server that might allow us either to load-shed or provide backpressure if we can't write to this file fast enough to keep up with requests.   However, I don't think that is required at this time as it is still an in-development feature.

@stevendanna would love to chat about any gen_server optimizations you had in mind.

:+1: I think this is good to merge.

:+1:

A few thoughts: the description on this PR is great. I'd love to start seeing more descriptions like that of why make it into commit messages too. The description from the PR will make it down into the merge commit, but as a general musing. If I only saw the commit message it would take me a lot longer to decipher what was going on.
Some actions tests were removed. Why was that? It appeared care was taken to make sure Actions still functioned (and it should) so I'm confused why the tests were removed.
Overall, from a brief read through, code structure seems fine and I'm :+1:. This seems like a great candidate to do a code walk through with someone from the chef server team on, though. It would help spread knowledge and I think would spur discussion around design that a PR like this tends to miss.

The description from the PR will make it down into the merge commit, but as a general musing.

I don't think that is true.  Only the summary line ends up in the merge commit.

@stevendanna Huh. Be good to confirm that for future reference, but I wouldn't hold up this PR over it. We can adjust as we go.

Re: removal of the tests - There were two pairs of tests that were essentially duplicates of each other.  The difference between the tests in the pairs was a set_env that was never added in sys.config. As a part of this PR, the env var check was removed (since it wasn't used) and consequently the env toggle in the tests wasn't needed.
TL;DR defunct test cruft was removed due to tested code path removal.

@elliott-davis :+1: Appreciate the updates and explanations.

Approaching this a different way from insights. Closing.


Quick notes/ Cheat Sheet -> C++

Description
A crisp and concise revision sheet to refer to while preparing for the Interview!
-> DS Implementation
-> Inbuilt functions
-> Important concepts
Domain
Interview Prep
Type of Contribution
Video, Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign document

@vishnu kindly close one of the 3 issues assigned to you or unassign to get this one assigned as others can work on this in the meanwhile!

/assign document

/unassign me

/assign document

Please assign this issue to me @syedareehaquasar.

Please go ahead Kanishk!

Okay thanks

Close this issue now !!


kubectl completion powershell doesn't work with CompletionPredictor

What happened: The CompletionPredictor module does not work with the kubectl completion script generated by the kubectl completion powershell command. Predictive completion does not work for kubectl commands.
What you expected to happen: Predictive completion should work for kubectl commands after importing the CompletionPredictor module and generating the kubectl completion script.
How to reproduce it (as minimally and precisely as possible):

Import the CompletionPredictor module with Import-Module -Name CompletionPredictor
Set the PredictionSource option to HistoryAndPlugin with Set-PSReadLineOption -PredictionSource HistoryAndPlugin
Generate and execute the kubectl completion script with kubectl completion powershell | Out-String | Invoke-Expression
Type a kubectl command and try to use predictive completion.

Anything else we need to know?: None.
Environment:

Kubernetes client and server versions (use kubectl version): Client Version: v1.25.4
Cloud provider or hardware configuration: N/A
OS (e.g: cat /etc/os-release): Windows 10

I think we should track this upstream in cobra.
https://github.com/spf13/cobra/issues/1949
/close


#15440 Migrate Ship Engine endpoint to package

Pull-request for issue https://github.com/slingr-stack/platform/issues/15440 created by @pasaperez-slingr
What it does?
Creation of the shipengine api package. Equivalence to what would be the endpoint.
How to test it?
Register the package on developer portal and from one app install and make some request according to the api.
Implementation notes
Create a the tag v2.0.0 and push.
Deployment notes
Register on local, idea2, tarzan and later on prod.

Did you run the tests? Do you have credentials to share? How can I test the package if a tag hasn't been created yet?

You can test with v1.0.0 tag, the package has been registered on idea2 on test1.
And the code with this endpoint:
log(JSON.stringify(pkg.shipengine.api.get('/labels')))


Better Annotation support

not depend on 3rd lib
bootstraper | booter | boot -> package definition
eq.

Bootstrap (package1.sub.sub, package2.sub.sub)

banchemark?

I think it's little bit outdated.


Hiera support

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
http://build-eu-00.elasticsearch.org//job/Puppet_Elasticsearch_PR/2/
Test FAILed.

I've enabled the hiera support for rspec-puppet but getting weird errors.
this is one of the errors:
  1) elasticsearch on redhat-5-x86_64 main class tests should compile into a catalogue without dependency cycles
     Failure/Error: it { should compile.with_all_deps }
       error during compilation: Parameter user failed on Exec[install_plugin_mobz/elasticsearch-head]: Only root can execute commands as other users at /home/richard/projects/puppet/modules/puppet-elasticsearch/spec/fixtures/modules/elasticsearch/manifests/plugin.pp:116
     # ./spec/classes/000_elasticsearch_init_spec.rb:42:in `block (5 levels) in <top (required)>'

Any idea's ?

Ugh. Yes.
Now you're creating an instance, your catalogue contains the Exec class for installing plugins.
You're specifying which user the Exec should run as which is something only root can do. As you're not root, the manifest won't compile.
But it's only a unit test so it shouldn't care, right?
A couple of other people have noticed this, see:-
https://tickets.puppetlabs.com/browse/PUP-2076
https://github.com/rodjek/rspec-puppet/issues/175
So I think it's a bug - I'm not really sure what the solution though I'm afraid...

I have removed the check for compiling and works now :-) stupid stuff :-1:

Only issue i'm running into now is that existing tests fail because the hiera config adds stuff to the params stuff that i'm not expecting in the tests. Any idea's?
I thought of adding a fact for the hostname and use that in the hiera config to ensure its only enabled then.
Any idea's ?

Yeah that makes sense.
You don't even need to have the custom fact be 'hostname', you can have something more descriptive along the lines of 'hiera_test_data' or whatever so it's more obvious what it's doing?
As long as you include it in the hiera.config hierarchy it will work just the same. You could then make the .yaml filename match or at least relate to the scenario or the data in the file? single_hiera_instance.yaml, multiple_hiera_instance.yaml, etc.
What do you think?

Oh that's a good one, forgot i can set a custom fact in there.
Thanks for the tip!

Got the main parts working to test out the instances and plugins parts.
Do you have any idea's on testing plugins_hiera_merge and instances_hiera_merge ?
All i can think of is adding an other custom fact that loads up an other hiera file and then use that?

I've just added some hiera spec tests.
Could you have a quick look if it covers most of it?
Cheers.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://build-eu-00.elasticsearch.org//job/Puppet_Elasticsearch_PR/20/
Test PASSed.

Looks good to me! I like the defaultinstance combined with scenario.

Thanks. Now i have to think up something for the acceptance testing with beaker.
Especially the differences between foss and PE make it a bit harder to manage the hiera.yaml file in a decent way.

Managed to add some simple acceptance testing as well.
Going to see if it wants to bake in CI decently before merging it.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
http://build-eu-00.elasticsearch.org//job/Puppet_Elasticsearch_PR/21/
Test FAILed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://build-eu-00.elasticsearch.org//job/Puppet_Elasticsearch_PR/23/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://build-eu-00.elasticsearch.org//job/Puppet_Elasticsearch_PR/24/
Test PASSed.

Extremely excited that we will not have to manage this outside of this module. Thanks so much for all the work here!


[BUG] Sub-grid command is not found when it is in the overflow menu

Bug Report
EasyRepro Version

[x] Microsoft Dynamics 365 Online Version 9.1 (9.1.x) (DB 9.1.x) online

UCI or Classic Web

[x] UCI (XrmApp)

Online or On Premise

[x] Online

Browser

[x] Chrome

Describe the bug
When a sub-grid command is in the overflow menu, ClickSubGridCommand() is unable to find it.
Code to reproduce
XrmApp.Entity.SubGrid.ClickCommand(subGridName, commandName);

-->
Expected behaviour
The sub-grid overflow menu button should be clicked followed by the command in the overflow menu.
Screenshots

Attempting to click a command other than the three shown will result in the exception:

No command with the name '' exists inside of '' CommandBar.

Additional context
I believe the following code to be the issue -
//Is the button in More Commands Overflow?
if (items.Any(x => x.GetAttribute("aria-label").Equals("More Commands", StringComparison.OrdinalIgnoreCase)))
{
    //Click More Commands
    items.FirstOrDefault(x => x.GetAttribute("aria-label").Equals("More Commands", StringComparison.OrdinalIgnoreCase)).Click(true);
    driver.WaitForTransaction();
...

When I inspected the HTML for the sub-grid, the aria-label attribute value was More commands for <entity display name> rather than More Commands. Perhaps More Commands is for a control that isn't a standard read-only grid? I would suggest changing this to a StartsWith check to avoid a breaking change.

@ewingjm - This has been fixed in the develop branch. It should be merged into release branch tomorrow as well as a NuGet update.
Thanks,
Tyler


import-certs.sh does not use the same config.sh paths that letsencrypt.sh does.

As evidenced here:
https://github.com/lukas2511/letsencrypt.sh/blob/d322e072d643dcee63785be1742a56a8178f74c0/import-certs.sh#L14
The actual paths checked by letsencrypt.sh:
https://github.com/lukas2511/letsencrypt.sh/blob/0e92aba206a60e00077d4b49e9479136f65587dc/letsencrypt.sh#L29

I really don't like that this is a separate script. Is there a reason this shouldn't be handeld with an "--import-certs / -i" option command in the main script?

I really don't like that this is a separate script. Is there a reason this shouldn't be handeld with an "--import-certs / -i" option command in the main script?

Please don't include helper scripts into the main script. I think the main reason why people are / will be using letsencrypt.sh is because it's simple and straightforward.

I agree with digint here. Other than copying the whole configuration file searching block, I can see two ways, one really "simple" and the other not that complex that would make it find the proper config file.

Getting CONFIG_DIR from environment CONFIG_FILE="${CONFIG_FILE:-${SCRIPTDIR}/config.sh}" which is unwieldy, but would allow the configuration file to be found anywhere.
Asking the main script for the config file. The import script already know about SCRIPT_DIR, which means it know where letsencrypt.sh is. It could source the script using a particular command (--source?) which would basically "do nothing", which makes the environment available in the script that sourced it.
This approach is cleaner, but does not allow passing --config to the import script. With care, part of the letsencrypt.sh's arguments parsing could work through the sourcing, but I'm not sure how clean and how easy it would be,

I agree with samueldr and digint. It's much cleaner to have an external helper script. Especially since that helper script will have to have to parts when deploying to a secondary machine (as digint did in his pull request, which I very much like).

Fixed with 1ab6a436c0c8b3588a6a398f32dcafa14560700b and 62037de04169ca36c49c581c37d0cffbce86e401.


remove epidemic week formatting requirements for CDF category values

I just noticed that the schema requires that the CDF values are either numeric or are strings in the format of an epidemic week, e.g. "EW202204".  I don't think there's a reason to impose this formatting requirement, though -- one could imagine collecting CDF predictions for an ordinal categorical variable other than epidemic weeks with strings naming the categories that don't match that format.
For example, looking here: https://github.com/Infectious-Disease-Modeling-Hubs/schemas/blob/2e79b13d309168824bcca7ef716ebeb9aa3cda44/v3.0.0/tasks-schema.json#L948-L953

Quick question, we want min and max removed also right? https://github.com/hubverse-org/schemas/blob/2e79b13d309168824bcca7ef716ebeb9aa3cda44/v3.0.0/tasks-schema.json#L951-L952

yeah, i agree that it makes sense to remove those too


A $network exception rule disables HTTPS filtering on some websites

Steps to reproduce

Enable AdGuard Base filter or add this rule to user filter:

@@<IP_ADDRESS>:443$network

Go here - https://www.nfmovies.com/
Check if there is AdGuard certificate on this website

Screenshots

Disable this rule - @@<IP_ADDRESS>:443$network
Restart web browser
Go to https://www.nfmovies.com/ and then check certificate

Screenshot

Expected behavior

@@<IP_ADDRESS>:443$network rule should not disable HTTPS filtering on https://www.nfmovies.com/
Actual behavior

@@<IP_ADDRESS>:443$network rule disables HTTPS filtering on https://www.nfmovies.com/
Screenshots

Your environment

Chrome 91
AdGuard for Windows 7.7 nightly 2 (build 3592, CL 1.7.227)

This is expected behavior - $network whitelist rules are used for bypassing everything.

What <IP_ADDRESS> has to do with that website?

Name:	www.nfmovies.com
Address: <IP_ADDRESS>

Ehm, that's unexpected. How in the world does it work?

So what should be done in case of this issue - https://github.com/AdguardTeam/AdguardFilters/issues/84833#issue-910667338?
Should I just close it and tell users that it will be not fixed or just write a comment to disable specific rule or make this rule @@<IP_ADDRESS>:443$network MacOS/Safari specific?
https://github.com/AdguardTeam/CoreLibs/issues/598
https://github.com/AdguardTeam/AdguardFilters/commit/6f02fbad0896f72e5c49b7a726b57e1ca4b83eae

@AdamWr do we really need that $network rule? Can it be disabled?

@AdamWr as I recall, we needed that rule for some rather old version of AG for Mac and it should work okay now.

Okay, I have commented out these rules - https://github.com/AdguardTeam/AdguardFilters/commit/c56b35321724267e392280fa4b56bd41dbb03f36

@AdamWr I am 100% sure you can safely remove them, just checked it with AG for Mac.

Done - https://github.com/AdguardTeam/AdguardFilters/commit/773e82a42b531dc9670133ebbe069103824439f3


Musicbot not working on linux server

2017-09-27T21:29:10.568826+00:00 app[worker.1]: Traceback (most recent call last):
2017-09-27T21:29:10.568858+00:00 app[worker.1]:   File "bot.py", line 15, in
2017-09-27T21:29:10.569026+00:00 app[worker.1]:     from modules.player import *
2017-09-27T21:29:10.569057+00:00 app[worker.1]:   File "/app/modules/player.py", line 5, in
2017-09-27T21:29:10.569193+00:00 app[worker.1]:     from lib.musicbot.bot import MusicBot
2017-09-27T21:29:10.569197+00:00 app[worker.1]:   File "/app/lib/musicbot/init.py", line 1, in
2017-09-27T21:29:10.569338+00:00 app[worker.1]:     from .bot import MusicBot
2017-09-27T21:29:10.569341+00:00 app[worker.1]:   File "/app/lib/musicbot/bot.py", line 39, in
2017-09-27T21:29:10.569502+00:00 app[worker.1]:     load_opus_lib()
2017-09-27T21:29:10.569519+00:00 app[worker.1]:   File "/app/lib/musicbot/opus_loader.py", line 18, in load_opus_lib
2017-09-27T21:29:10.569672+00:00 app[worker.1]:     raise RuntimeError('Could not load an opus lib. Tried %s' % (', '.join(opus_libs)))
2017-09-27T21:29:10.569682+00:00 app[worker.1]: RuntimeError: Could not load an opus lib. Tried libopus-0.x86.dll, libopus-0.x64.dll, libopus-0.dll, libopus.so.0, libopus.0.dylib
I am getting this error when trying to run the bot on a linux server. What can i do to fix this?

Try installing the package libopus-dev (via apt-get or yum, depending on your flavor of linux)

The above response should fix this. This is mentioned in our Linux guide.


ImportError: cannot import name '_tiktoken' from partially initialized module 'tiktoken'

The bug
It seems there's some circular import problem with tiktoken. Running just the code below, with no additional imports, I'm getting the error below.
To Reproduce
import guidance
gpt4 = guidance.llms.OpenAI("gpt-4")

---------------------------------------------------------------------------
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/me/superscan/env/lib/python3.11/site-packages/guidance/llms/_openai.py", line 177, in __init__
    import tiktoken
  File "/Users/me/superscan/env/lib/python3.11/site-packages/tiktoken/__init__.py", line 1, in <module>
    from .core import Encoding as Encoding
  File "/Users/me/superscan/env/lib/python3.11/site-packages/tiktoken/core.py", line 9, in <module>
    from tiktoken import _tiktoken
ImportError: cannot import name '_tiktoken' from partially initialized module 'tiktoken' (most likely due to a circular import) (/Users/me/superscan/env/lib/python3.11/site-packages/tiktoken/__init__.py)

System info (please complete the following information):

OS (e.g. Ubuntu, Windows 11, Mac OS, etc.): Mac OS 13.3.1
Guidance Version (guidance.__version__): 0.0.64

Resolved by switching from vanilla Pip to Poetry, likely a dependency conflict


be able to set custom spinner & custom loading bar templates

hrr, already doable by using the propertys ;D


Camera is not responding

Hi, I am using this module........ Camera is not responding for Android and iOS.
here is my screenshot:

closing this due to lack of activity


[FLINK-5869] [flip-1] Introduce abstraction for FailoverStrategy

This PR has two sets of changes that I could not pull apart into separate pull requests.
(1) Termination Futures
Prior to this change, the ExecutionGraph decided when cancellation and finishing was complete by tracking how many ExecutionJobVertex were in a terminal state.
This abstraction is too inflexible to track when subregions of the graph are in a terminal state. To fix that, this change introduces a termination future on the Execution. Building conjunct futures of the termination futures, any observer can track when any number of vertices in a terminal state.
The ExecutionGraph now also uses that model to track when cancellation of all vertices during failover is complete.
Local Failover and FailoverStrategy
The ExecutionGraph now supports local failover and global failover. Quoting from the JavaDocs:

Global failover aborts the task executions for all vertices and restarts whole data flow graph from the last completed checkpoint. Global failover is considered the fallback strategy that is used when a local failover is unsuccessful, or when a issue is found in the state of the ExecutionGraph that could mark it as inconsistent (caused by a bug).

Local failover is triggered when an individual vertex execution (a task) fails. The local failover is coordinated by the FailoverStrategy. A local failover typically attempts to restart as little as possible, but as much as necessary.

Between local- and global failover, the global failover always takes precedence, because it is the core mechanism that the ExecutionGraph relies on to bring back consistency. The guard that, the ExecutionGraph maintains a global modification version, which is incremented with every global failover (and other global actions, like job cancellation, or terminal failure). Local failover is always scoped by the modification version that the execution graph had when the failover was triggered. If a new global modification version is reached during local failover (meaning there is a concurrent global failover), the failover strategy has to yield before the global failover.

Failover Strategies
How exactly local failover happens is the concern of a pluggable FailoverStrategy.

The default failover strategy simply triggers a global failover
The pull request introduces a very simple restart individual failover strategy that restarts tasks without any connections to other tasks independently.

Tests
This pull requests adds new tests for

The termination future abstraction
The global mod version handling
Proper handling of concurrent local- and global failover

The pull requests rewrites various original tests. This was necessary, because the tests were using Mockito very heavily and re-building or whitebox testing specific behavior that was affected by the changes.
The changes to the tests introduce simple ways to actually bring up a functional ExecutionGraph and walk it through its state transitions. That way the tests now rely minimally on mocking and actually test the proper ExecutionGraph, rather than a mock which is expected to behave similar to the proper class.

Manually merged in 8ed85fe49b7595546a8f968e0faa1fa7d4da47ec


Feature request: add an option to include time zone in git commit message

May you please provide more information?
What is your use case?
Can you give an example?

Hi!
The current commit message when auto-committing using Obsidian Git is vault message: YYYY-MM-DD HH:mm:ss. I don't know if the plugin can also access the current timezone/locale of the device you're currently using, or if there is a way to include timezone in the timestamp. Something like:
vault message: 2023-02-04 13:09 +0800
Not that very important since system git already captures timezones when committing (as it shows in the git log) but it would be useful to also see the timezone in the commit message at a glance in git repo history. This is also useful when a user travels often and changes timezone quite a lot.
I hope this helps. Thank you!

That should be possible via the formatter list here. ZZ should do what you want

This works. Thank you very much!


IPV6 does not work during docker build command

During a docker build communication over IPv6 via fe80::1%eth0 does not work to the host machine. The built container however functions correctly. We can see that correctly ipv6 link local addresses do exist in /proc/net/if_inet6.
List the steps to reproduce the issue:

From a fedora:latest with docker run -i -t /bin/bash:
[root@c76f2b9d2d77]/# nmap <IP_ADDRESS> -p 3128

Starting Nmap 6.47 ( http://nmap.org ) at 2015-10-11 03:11 UTC
Nmap scan report for <IP_ADDRESS>
Host is up (0.000065s latency).
PORT     STATE SERVICE
3128/tcp open  squid-http
MAC Address: 02:42:FC:D5:D8:71 (Unknown)
Nmap done: 1 IP address (1 host up) scanned in 0.13 seconds
[root@c76f2b9d2d77]/# nmap -6 fe80::1%eth0 -p 3128
Starting Nmap 6.47 ( http://nmap.org ) at 2015-10-11 03:11 UTC
Nmap scan report for gateway (fe80::1)
Host is up (0.00017s latency).
PORT     STATE SERVICE
3128/tcp open  squid-http
MAC Address: 02:42:FC:D5:D8:71 (Unknown)
Nmap done: 1 IP address (1 host up) scanned in 0.15 seconds
2:
Using the following Dockerfile:
FROM fedora:latest
RUN dnf install -y nmap
RUN cat /proc/net/if_inet6
RUN nmap <IP_ADDRESS> -p 3128
RUN nmap -6 fe80::1%eth0 -p 3128
With docker build -t test .
Describe the results you received:
docker build -t testcontainer:latest .
Sending build context to Docker daemon 2.048 kB
Step 0 : FROM fedora:latest
---> ded7cd95e059
Step 1 : RUN dnf install -y nmap
---> Running in<PHONE_NUMBER>3d
Last metadata expiration check performed 0:00:19 ago on Sun Oct 11 03:15:20 2015.
Dependencies resolved.
Package           Arch           Version                 Repository       Size
Installing:
libpcap           x86_64         14:1.7.3-1.fc22         updates         149 k
nmap              x86_64         2:6.47-2.fc22           fedora          4.0 M
nmap-ncat         x86_64         2:6.47-2.fc22           fedora          135 k
Transaction Summary
Install  3 Packages
Total download size: 4.3 M
Installed size: 17 M
Downloading Packages:
Total                                           233 kB/s | 4.3 MB     00:18
Running transaction check
Transaction check succeeded.
Running transaction test
Transaction test succeeded.
Running transaction
Installing  : libpcap-14:1.7.3-1.fc22.x86_64                              1/3
warning: Unable to get systemd shutdown inhibition lock
Installing  : nmap-ncat-2:6.47-2.fc22.x86_64                              2/3
Installing  : nmap-2:6.47-2.fc22.x86_64                                   3/3
Verifying   : nmap-2:6.47-2.fc22.x86_64                                   1/3
Verifying   : nmap-ncat-2:6.47-2.fc22.x86_64                              2/3
Verifying   : libpcap-14:1.7.3-1.fc22.x86_64                              3/3
Installed:
libpcap.x86_64 14:1.7.3-1.fc22            nmap.x86_64 2:6.47-2.fc22
nmap-ncat.x86_64 2:6.47-2.fc22
Complete!
---> aa84cdbacaed
Removing intermediate container<PHONE_NUMBER>3d
Step 2 : RUN cat /proc/net/if_inet6
---> Running in b6c8b1e812cb
00000000000000000000000000000001 01 80 10 80       lo
fe8000000000000000000242ac11000c bd 40 20 c0     eth0
fe800000000000000042acfffe11000c bd 40 20 c0     eth0
---> c5483d1e5e0b
Removing intermediate container b6c8b1e812cb
Step 3 : RUN nmap <IP_ADDRESS> -p 3128
---> Running in 89dab5398379
Starting Nmap 6.47 ( http://nmap.org ) at 2015-10-11 03:16 UTC
Nmap scan report for <IP_ADDRESS>
Host is up (0.000053s latency).
PORT     STATE SERVICE
3128/tcp open  squid-http
MAC Address: 02:42:FC:D5:D8:71 (Unknown)
Nmap done: 1 IP address (1 host up) scanned in 0.13 seconds
---> fc208785eba9
Removing intermediate container 89dab5398379
Step 4 : RUN nmap -6 fe80::1%eth0 -p 3128
---> Running in f2c4bf3b8a1c
Starting Nmap 6.47 ( http://nmap.org ) at 2015-10-11 03:16 UTC
Failed to convert source address to presentation format!?!  Error: Address family not supported by protocol
QUITTING!
The command '/bin/sh -c nmap -6 fe80::1%eth0 -p 3128' returned a non-zero code: 1
Describe the results you expected:
IPv6 to work during docker build
Provide additional info you think is important:
This is not an interpretation issue with the Dockerfile and the % or other chars, as I picked this up after using an ADD file command to add a local proxy via ipv6 to a container. The following command that relied on the proxy then failed due to ipv6 not working.
In my case I am checking port 3128, but this could easily be anything on the host machine.
docker version
Client:
Version:      1.8.2-fc22
API version:  1.20
Package Version: docker-1.8.2-1.gitf1db8f2.fc22.x86_64
Go version:   go1.4.2
Git commit:   f1db8f2/1.8.2
Built:
OS/Arch:      linux/amd64
Server:
Version:      1.8.2-fc22
API version:  1.20
Package Version:
Go version:   go1.4.2
Git commit:   f1db8f2/1.8.2
Built:
OS/Arch:      linux/amd64
docker info
Containers: 5
Images: 73
Storage Driver: btrfs
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 4.3.0-0.rc4.git0.1.mbpv2.fc24.x86_64
Operating System: Fedora 22 (Twenty Two)
CPUs: 8
Total Memory: 15.46 GiB
Name: franky.ipa.example.com
ID: WGE2:VZWO:5DPK:4AV7:BVUB:2JGD:TVHN:S7HP:6AGD:CJ56:QMRJ:JSC4
Debug mode (server): true
File Descriptors: 25
Goroutines: 56
System Time: 2015-10-11T13:15:56.901268426+10:00
EventsListeners: 0
Init SHA1: 603c4f89aedd044c26f9b65e8f6af985190a4da3
Init Path: /usr/libexec/docker/dockerinit
Docker Root Dir: /var/lib/docker
Linux franky.ipa.example.com 4.3.0-0.rc4.git0.1.mbpv2.fc24.x86_64 #1 SMP Fri Oct 9 13:04:14 AEST 2015 x86_64 x86_64 x86_64 GNU/Linux

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

Hey @Firstyear, I currently have an issue with similar signs. It looks like docker had run the command you passed within the Dockerfile, before IPv6 network was established.
Could you check this two scenarios based on your description from the issue:
1. Run nmap right after container was started:
› docker run -i --rm fedora:latest nmap -6 fe80::1%eth0 -p 3128

2. Wait a little before running nmap
› docker run -i --rm fedora:latest sh -c 'sleep3; nmap -6 fe80::1%eth0 -p 3128'

My guess here is that everything will work as expected in the second case.
/cc @thaJeztah could you suggest anyone who could look at the issue?

probably;
ping @aboch @mavenugo or @mrjana should be able to have a look :)

Some additional random thoughts I currently have: my host's /etc/resolv.conf has only one DNS record (which comes from my cloud provider).
If I run:
› docker run ... bash -c 'strace -e poll,select,connect,recvfrom,sendto nc mirror.yandex.ru 80'

strace's log shows that there is a timeout in DNS resolve:
···
connect(3, {sa_family=AF_INET6, sin6_port=htons(53), inet_pton(AF_INET6, "2a02:6b8:0:3400::1023", &sin6_addr), sin6_flowinfo=0, sin6_scope_id=0}, 28) = 0
poll([{fd=3, events=POLLOUT}], 1, 0)    = 1 ([{fd=3, revents=POLLOUT}])
poll([{fd=3, events=POLLIN}], 1, 5000)  = 0 (Timeout)
poll([{fd=3, events=POLLOUT}], 1, 0)    = 1 ([{fd=3, revents=POLLOUT}])
poll([{fd=3, events=POLLIN}], 1, 5000)  = 0 (Timeout)

But if I add a small sleep as the first command, DNS will be resolved as expected:
› sudo docker run --rm -i test1 bash -c 'sleep 2; strace -e poll,select,connect,recvfrom,sendto nc mirror.yandex.ru 80'

···
connect(3, {sa_family=AF_INET6, sin6_port=htons(53), inet_pton(AF_INET6, "2a02:6b8:0:3400::1023", &sin6_addr), sin6_flowinfo=0, sin6_scope_id=0}, 28) = 0
poll([{fd=3, events=POLLOUT}], 1, 0)    = 1 ([{fd=3, revents=POLLOUT}])
poll([{fd=3, events=POLLIN}], 1, 5000)  = 1 ([{fd=3, revents=POLLIN}])
recvfrom(3, "\336\\\201\200\0\1\0\0\0\1\0\0\6mirror\6yandex\2ru\0\0\1"..., 2048, 0, {sa_family=AF_INET6, sin6_port=htons(53), inet_pton(AF_INET6, "2a02:6b8:0:3400::1023", &sin6_addr), sin6_flowinfo=0, sin6_scope_id=0}, [28]) = 108
poll([{fd=3, events=POLLIN}], 1, 4996)  = 1 ([{fd=3, revents=POLLIN}])

I'm pretty sure that everything is file with this named server as when I add the second host to the resolv.conf  (no matter where: at the top or at the bottom of the list) the same nc call works only with second DNS host from the list of nameservers.
E.g. with resolv.conf
nameserver A
nameserver B

or
nameserver B
nameserver A

the resolve call to the first nameserver will alway be timedout.

Thanks for reporting @Firstyear  this could be the same as this issue; https://github.com/docker/docker/issues/18871, which should be resolved by https://github.com/docker/docker/pull/19198,and is part of the docker 1.10.0 release.
If you have a system available for testing (not on production yet), would you be able to give the current release candidate a spin and see if this is resolved for you? https://github.com/docker/docker/releases/tag/v1.10.0-rc1 please note that the 1.10 release uses a content-addressable storage, which requires a migration step the first time docker is run. Depending on the number of images and containers on your host, this migration may take a long time to complete. Read more on this in the wiki https://github.com/docker/docker/wiki/Engine-v1.10.0-content-addressability-migration

ping @aboch can you have a look, do you think this is resolved by https://github.com/docker/docker/pull/19198?

ping @Firstyear @narqo are you still having this issue on docker 1.10 or 1.11? Can we close this?

Seems to be fixed on 1.10 from what I can see.


Deno install should add to your bash/zsh config on first run if DENO_PATH is missing

Hello! To get Deno install working I had to manually add it to my zsh config. It would be great if this was done automatically.

This was tried multiple times before (#52 #157 #177 #178), but it's difficult and complex to detect the users' environment correctly. We rather prefer to keep the install script simple.

Ah ok no worries, that's fair enough. Just one extra piece of friction/step to get new users onto Deno though.


keycloak: 21.1.1 -> 21.1.2

Diff: https://github.com/keycloak/keycloak/compare/None...21.1.2
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.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

[x] Fits CONTRIBUTING.md.

@NickCao It's not building on Linux, I just noticed.

It builds but is failing test.

I guess we don't merge it now then?

I'm unable to reproduce the failure locally, let's try again.
@ofborg test keycloak.mariadb keycloak.mysql keycloak.postgres

It passes, so another flaky test....

It passes, so another flaky test....

Linux tests are still broken here.

It passes, so another flaky test....

Linux tests are still broken here.

Which one? Both x86_64-linux and  aarch64-linux looks fine to me.

Here https://github.com/NixOS/nixpkgs/pull/240297/checks?check_run_id=14617001272

It's from the previous run.

@ofborg eval

ofborg seems to refuse doing another eval, let me reopen another pr.


ipxe: fix build by not generating 64bit efi on 32bit systems

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

@GrahamcOfBorg build ipxe

@GrahamcOfBorg build ipxe


chore(pre-commit): enable commitlint

This change is part of the following stack:

#52

#53

#54

#55

#56

#57 ◀

Change managed by git-spice.


Docker updates in metanorma.com; closes https://github.com/metanorma/metanorma.com/issues/100

Thank you @opoudjis !


Update FAST 2022

Which conference does this PR update?

-FAST 2022
Related URL

-https://www.usenix.org/conference/fast22

Hi @ymao, we recommend to append the new info to the existing file. Please refer to this pr and fix it~


Move broadcast_to to cupy.core.core

This PR moves broadcast_to from cupy/manipulation/dims.py to cupy/core/core.pyx. This is necessary for methods inside of core to call broadcast_to.
Some changes being made include ...

the method is now defined by cpdef.
on line 1682 of core.pyx, j is changed to j % array.ndim. This is necessary because _strides does not accept negative values as index

LGTM!


Position Difference onPaneMove

@moklick I need a position difference from the default position (0,0) when the user has moved the pane so that I can calculate accurate node position

Hey @agarwal00s
can you exactly describe what you want to do? Why do you want to calculate node positions?

I am trying to create a node when we start a node connection and drop it on the canvas. The node position depends on where the user stopped (onConnectStop), so I'm getting the mouse event onConnectStop, but if the user has moved the pane then my node position has to be changed depending on how much the pane is moved, that's why I want this difference value.

If you want to get the internal position from a screen position you can use the project function that is part of the reactflowInstance you receive in the onLoad callback. This function is meant to be used for drag and drop handling.

Thanks @moklick it worked out.


Initial refactor: separate pool and provider

Summary of changes
Changes introduced in this pull request:

[] Added documentation (i.e. code comments)
[X] Separate the message pool provider from the message pool

Reference issue to close (if applicable)

Closes #1021
Other information and links

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


fix rendering with devicePixelRatio

Changes from @heshan0131
Fixes #1

Tested, picking also works fine.
Thanks! @heshan0131 @chrisirhc


[Accessibility][Usability]A11y_VSCode_ActivityBar_Keyboard: The tool tip is not appeared on avtivity bar button list

Environment Details:
VSCode Version : 1.24.0
OS Version : Win10
Additional Details:
Keyboard,Usability
Repro Steps:
1.Navigate to "Activity Bar" at left side of the screen after login.
2.Navigate to any item such as "Explorer,Search,SourceControl,Debug,Extensions,Setting" buttons.
3.Observed that keyboard focus is going on the button list but the tooltip is not appeared.
Actual:
Keyboard focus goes on the button list but tooltip is not appeared on it.
Expected:
if the button have tooltip then after keyboard focus tooltip should appeared on it.
Recommendations:
Refer below link which is repository of bug fixes code snippets.
https://microsoft.sharepoint.com/teams/msenable/mas/Pages/browse-fixes.aspx
User Impact:
Keyboard only user will not unserstand the extra information which is provided by the tooltip.
Attachment For Reference:
[Accessibility][Usability]A11y_VSCode_ActivityBar_Keyboard.pptx
Does this issue occur when all extensions are disabled?: Yes

@AccessibilityTestingTeam-TCS to clarify you mean that the tooltip should visually appear on each button/action/toolitem that gets keyboard focus?

Here,the issue is only by mouse over the tool tip get appeared.But if keyboard only user wants to see or know the extra information which is provided in tooltip then user will not get it.So without mouse over only by the keyboard focus tooltip should appeared.


KnockoutSpa (router.ts) Event listener and non trivial a tags

The code for the clickEventListener in router.ts does not capture the click event on a tags that contain child elements:
<a data-bind="attr: { href: url }"

Whereas the non tag version works fine.
<a data-bind="attr: { href: url }"

Switching to use the jQuery event selector and using currentTarget fixes everything for me. There's probably a Vanilla JS way of achieving the same thing but jQuery is in the project already and it is a schoolnight.
this.clickEventListener = evt => {
    let target: any = evt.currentTarget;
    if (target && target.tagName === 'A') {
        let href = target.getAttribute('href');
        if (href && href.charAt(0) == '/') {
            history.push(href);
            evt.preventDefault();
        }
    }
};

$(document).on('click', 'a', this.clickEventListener);

Thanks for reporting this! I'll adapt the template to do something along these lines soon.

Thanks again - this is now in the template source.


OnChange Event

Is your feature request related to a problem? Please describe.
I'm trying to implement an auto-save feature but the vue-ckeditor2 only has an onblur() event. If my user doesn't leave the editor control before closing the window their changes will be lost.
Describe the solution you'd like
Can you add an "onchange()" event or is there simply another way to handle this with the existing vue-ckeditor?
Describe alternatives you've considered
No known alternatives at this time.
Additional context
N/A

I decided to use a watch on the same model I'm using for the editor and that's working fine.


enable lanching vnc with a user account

This allows a more common user experience with a non-root user. You can use the usual --user and --home extensions.

With https://github.com/osrf/rocker/pull/138 landed. I'm going to merge this as we're actively testing with it.


IMU calibration loading

Autoloading IMU calibration data does not improve the IMU calibration status. Calibration saving appears to work fine so the problem is likely in loading the saved calibrations when initializing the IMU.

Write unit tests to determine whether or not the IMU successfully calibrates and or loads stored calibration data. Also checks to see if the IMU's calibration settings are sucessfully stored to the imu calibration data file.


Return form submitting when input field is not valid

Hi, I used this:
bootstrapValidate(    '#docs-demo',    'email:Enter a valid E-Mail Address!' );
bootstrapValidate(    '#docs-demo',    'required:Please fill out this field!' );
And it works, It will display invalid messages
But return form submitting too,
how i can fix it ?
thanks

What do you mean by "But return form submitting too?"


Fixes to Emscripting a C library to Wasm

What's changed, or what was fixed?
In the Emscripting a C library to Wasm article,

Added missing commas.
Removed an extra period at the end of a sentence.

CC: @petele

:tada: This has been pushed live to https://developers.google.com/web/


If the first ExtendTips hash is bad, discard it and re-check

The clippy errors are unrelated to this PR, see #994.

I think rather than using split_first you might be able to use slice patterns to handle this

That worked well, see 1bcfca6.

Wouldn't it be simpler just to discard these responses as before? We already do fanout so that extracting useful data from each individual response is not important.

Wouldn't it be simpler just to discard these responses as before? We already do fanout so that extracting useful data from each individual response is not important.

I tried increasing fanout to deal with these failures, but increasing fanout on the testnet caused repeated failures.
I think that's because we were querying most of the available peers during fanout. Because we query those peers, zcashd tries to be helpful, and sends an additional unsolicited inv message when it thinks we're finished with the last one.
But Zebra can misinterpret that inv message as a response to our next query. And because Zebra uses checked tips, the inv responses are misaligned by at least one block.
On testnet, because there aren't many peers, all of the fanout peers would return an unsolicited response reasonably often.
So I think the fanout number is a tradeoff between:

choosing enough peers to get at least one good response, most of the time, and
choosing a small enough proportion of the peers that we leave some fresh peers to answer future queries.

There may be other changes that help achieve these goals, but I don't understand the code well enough to make them. (And we don't have the necessary tests to verify them.)

Since each sync failure resets state, and syncing starts with ObtainTips, each failure makes the next failure more likely.
Because of these repeated failures, each triggering the next failure, Zebra can get into a state where it is only failing, and never making progress.


Clear method

I would like it if the CuckooFilter had a .clear() method which deleted all the entries. This would be useful to me when doing kNN graph search (HNSW) to prevent reallocating memory on each nearest neighbor search. This can be a problem when dealing with small search spaces since a lot of the time can be taken up on just allocation since little searching actually needs to be done.
Is this of interest? I assume it should be a small change. Just looking to see if this kind of change would be accepted.

+1 interested in creating a PR


fix: added the right class for the selector used

Description
A failing test was introduced with fix in #19994 where the class name was updated but the cypress selector was not updated
Fixes #20171
Type of change

Chore (housekeeping or task changes that don't impact user perception)

How Has This Been Tested?

Cypress

made a change to the selector adding the right class to it

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=3634319

The above reported issue was resolved on re-running
The initial error was


Broken link on readme.md

Brief description of your issue
At main readme.md, under subheading "Automatically create a rest source" the link to powershell New-WinGetSource is broken.
https://github.com/microsoft/winget-cli-restsource#automatically-create-a-rest-source

Thank you @dahovey for the feedback. The following PR has been submitted and is awaiting approval to correct the links.
https://github.com/microsoft/winget-cli-restsource/pull/106


POST response base64

Example
class Base64(BaseModel):
    base64_str: str
    photo_name: str

@app.post("/base64")
async def post_base64Image(b64: Base64):
    imgstr = b64.base64_str
    imgdata = base64.b64decode(imgstr)
    filename = '%s.png' % b64.photo_name
    with open(filename, 'wb') as f:
        f.write(imgdata)
    photo_id = 0
    paths = int(photo_id / 1000) * 1000
    with open(r"E:\images1024x1024\%05d\%05d.png" % (paths, photo_id), "rb") as img_file:
        b64_output = base64.b64encode(img_file.read())
    return {"Image" : b64_output.decode('utf-8')}

@app.post("/response_model/", response_model=Base64)
async def response_models(b64: Base64):
    photo_id = 0
    paths = int(photo_id / 1000) * 1000
    with open(r"E:\images1024x1024\%05d\%05d.png" % (paths, photo_id), "rb") as img_file:
        b64_output = base64.b64encode(img_file.read())
    b64.base64_str = b64_output.decode('utf-8')
    return b64

Description

If I use this two HTTP POST to send base64 image with size 64*64 is ok.

But when I want to send 1024*1024 it just doesn't work.

The client side is using swift to call the POST method.

How could I send 1024*1024 base64 to it?

Or is my POST funtion is not well defined?

Environment

macOS
FastAPI Version: 0.61.1
Python version: 3.8.6

What error are you getting in your server console?

Not show any error, but just show nothing.

If you can provide us an example that doesn't depend on your local path, some image samples and explain how we can reproduce the same behavior, it will be very cool. :sunglasses: :+1:


[PL-51751] SMP 0.17.1 patch RN

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes:  SMP 0.17.1 patch RN
Jira/GitHub Issue numbers (if any): PL-51751
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[x] Successful preview build.
[x] Code owner review.
[x] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6663206831a1e63b35aa48d3--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6672ff678132604e9c4712b4--harness-developer.netlify.app


flexget: 3.8.2 -> 3.8.3

Description of changes
change log: https://flexget.com/ChangeLog (just points to the diff)
diff: https://github.com/Flexget/Flexget/compare/v3.8.2...v3.8.3

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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/)
23.11 Release Notes (or backporting 23.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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 248269 run on x86_64-linux 1

2 packages built:

flexget
flexget.dist


update farm-ng-package

@ethanrublee i'm not very happy with this pattern but the way it is with setuptools, the Command class is very flexible to pass external params programmatically

I see. No worries, let's go with it. One question, can you put the option parameters into the setup.cfg and they automagically get set on these commands?


[Feature request] Automatically pause playback when an incoming phone call arrives

Can you please add a setting to automatically pause playback when a phone call arrives (and resume playback when the phone call ends)?
It would make your already great app even better :)
Thanks!

https://github.com/xbmc/Kore/pull/190

Oh, great, thanks! I browsed the issues to make sure I wouldn't make a duplicate request, but I didn't check the pull requests.

In latest version.


Add lxc help --sub-commands to list all sub-commands

The lxc command line tool has a lot of sub-commands at this point.
lxc help or lxc --help will show a subset of the top level ones (all if --all is passed).
It'd be nice for lxc help --sub-commands to show a tree of the sub-commands, so lxc help --all --sub-commands would show absolutely everything that's supported.

Hi. I'm a CS student at UT Austin and am currently taking a Virtualization class. I would be happy to take a look at this issue?

Sounds good, assigned it to you!

Should I include the descriptions along with the listed sub-commands or just their names?

I think having the one line description is quite useful.
I'm assuming this is all automated so we don't have to actually duplicate any of those strings, right?

Sounds good. Correct, none of the strings were duplicated. Since the help text is generated by CobraCommand, and CobraCommand doesn't have direct support for displaying sub-commands, but it does allow the user to override the default help template, so my approach was to create an alternate help template that includes sub-commands and is only used if the --sub-commands flag is passed.
I have a few padding issues I'm working out and then I should be good to put in the pull request.

Sounds good!


0.0.3

Auto-generated docs with Sphinx, available at https://viljarjf.github.io/QM_sim/
Refactors, move temporal solvers and Hamiltonian class
Add GPU support with PyTorch as an optional dependency
Adds plots
Adds tqdm to temporal solvers
WIP non-isotropic effective mass

Passes all examples and tests, except the solver comparison with C++-based solvers.
C++ compilation is removed (commented out in the CMakeLists file), but the files are not removed in case they become useful later.

Give me at least until the end of May and I may have time to look at it ;)

Might be an issue with docs generation with a clean environment, I think matplotlib needs to be added to the required modules

And the example code for spatial derivatives has been mangled

Implements #27
Implements #8


Color params in materials not showing color picker

When editing material paramaters i noticed that diffuse color and other color values are just shown as vec4 in imgui and not colors.
Id recommend changing it to a color picker. (i had a look but so far iv not been able to work out where it gets the imgui code from for inspector windows)

It's in editor project, MaterialInspectot or similar. You can open
InspectorTab, put a breakpoint on rendering method call and step in
On June 9, 2019 19:22:58 dragonCASTjosh<EMAIL_ADDRESS>wrote:

When editing material paramaters i noticed that diffuse color and other
color values are just shown as vec4 in imgui and not colors.
Id recommend changing it to a color picker. (i had a look but so far iv not
been able to work out where it gets the imgui code from for inspector windows)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

This was fixed a while back.


⚠ Jenkins Plugins Artifactory has degraded performance

In a99b879, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1192 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 43a8514 after 28 minutes.


Surface initialization errors when using happo-plugin-puppeteer

Ping @emilsjolander -- this addresses the issue we debugged together.


Integrate react-native-screens

Integrating react-native-screens may provide potential memory savings.

@osdnk maybe you have some thoughts on how to do this? the main problem is that we can't set the active prop based on the index, as you can still see the unfocused screens when swiping.

Closing since react-native-screens is buggy. Will revisit when it becomes more stable.


feat: Migrate Pigeon TTL to block height

Related Github tickets
https://github.com/VolumeFi/paloma/issues/707
Background
The previous Pigeon keep alive implementation would use a five minute TTL and compare against the current block time. This would frequently lead to problems after a chain halt, during which Pigeons would not be able to refresh their TTL.
This change removes the time based approach and substitutes the block time with block height, with a default of 185 blocks of lifetime before a Pigeon is considered stale.
Testing completed

[x] test coverage exists or has been added/updated
[x] tested in a private testnet

Breaking changes

[x] I have checked my code for breaking changes
[x] If there are breaking changes, there is a supporting migration.

On further review, I think we need to increment the required Pigeon version and have an associated Pigeon PR with this.
When Pigeon checks the keep alive, it's expecting a time back. https://github.com/palomachain/pigeon/blob/master/relayer/keep_alive.go#L13

That's true, but it still works as expected. Pigeons will still report in (due to the time stamp always being lower), it works without issues in test net. Although the frequency is elevated of course.
You're correct that we should handle this properly, but we'll need the Paloma update to go through first to reference the updated Proto definitions (unless we want to work against my feature branch, which I think is kind of messy), and if I increase the minimum Pigeon version now, we aim at a non-existing version.
So I'd rather follow up with that in a separate PR. What do you think?


normalize_comments is ignored inside rewrite_missing_comment

https://github.com/rust-lang-nursery/rustfmt/blob/d19fc450c06cd480ef2b9e1031b55ff10df080a7/src/comment.rs#L506
We should replace false with context.config.normalize_comments.

Sorry to go for the low hanging fruit here! Hopefully I'll be able to contribute a little more in future.


templates: Merge Help and API doc main into documentation_main.

https://github.com/zulip/zulip/issues/10061
Testing Plan:
I checked out both /api/ and /help/ to make sure that they both rendered appropriately.
GIFs or Screenshots:
No changes to the UI. This is just a backend refactoring

@Lyla-Fischer can you fix the linter error?

@Lyla-Fischer generally this looks great; I posted a couple quick comments.  Definitely nice to have this eliminate a bunch of duplicated code.
@eeshangarg FYI.

@eeshangarg can you help with debugging the JS part of this?  It appears that the HTML is being rendered correctly, but the ul entries for rest-endpoints.md are all being hidden.  My guess is that this merge is causing JS for collapsing inactive sections, that's only intended to run on the /help/ pages to also run on /api.

@timabbott, @Lyla-Fischer: Pushed an updated version of this PR, which fixes the sidebar issue. See #10434 :) Thanks!

Great, that works.  Closing in favor of #10434.


Controller GUI empty if with stuff on the network.

Using McJtyLib 1.1x-2.3.11 and Xnet 1.2.0 on Minecraft 1.10.2
I've got the Xnet controller connected in the back, and I've got an RFTools storage scanner, RFTools Control processor, and a drawer controller from storage drawers all hooked up to the network, but nothing's showing up in the controller, and I can't create or edit channels.

Show a screenshot of the setup (things around the controller)
Also logs please

latest.txt

I cannot see the xnet cables which is what I am interested in

Is there actually a connector attached to the controller? That's what I wanted to see

Hello
I've got the same pb..
My controller detect anything (chest, furnace...)
I've the same screen like the first post.
Please help
Thx
(Sorry for my english, I'm French :p)

What version of XNet?

Xnet : 1.1.2
With Compatlayer And Mcjtylib
Maybe i don't use the good version. If you can advise me for version of Xnet, Compatlayer and Mcjtvlib ? :)
PS : RFTOOLs works really good !

Just use the latest version of everything.  For xnet that is 1.3.1

Mcjtvlib don't work With the last version because i'm in 1.10.2
Minecraft ask me the version for 1.11 and 1.10 so i've got 2 Mcjtvlib in my mods document

Oh no. That's very wrong. The latest version of McJtyLib works fine on 1.10. Remove the other one. The only McJtyLib you should have is: 1.1x-2.3.11
Same for all my mods. Only use the versions starting with 1.1x. They are all for 1.10 and 1.11 at the same time


Plotting Data Using model.predict

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock
example script provided in TensorFlow): YES
OS Platform and Distribution (e.g.,
Linux Ubuntu 16.04): MacOS CATALINA
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if
the issue happens on mobile device:
TensorFlow installed from (source or
binary): BINARY
TensorFlow version (use command below): 2.1
Python version: 3.7.6
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from
source):
CUDA/cuDNN version: 10.2
GPU model and memory:

Describe the current behavior
I am attempting to plot the prediction values .  However predictions is 2-D and the data must be one dimensional.
Here is the predictions code:
#Get predicted price values
predictions = model.predict(x_test)
predictions = scaler.inverse_transform(predictions)
print(predictions.shape)

#get root mean squared error
rmse = np.sqrt(((predictions - y_test) ** 2).mean())
print(rmse)

#Plot the data
train = data[:training_data_len]
valid = data[training_data_len:]
valid['Predictions'] = predictions

#Visualize the data
plt.figure(figsize=(16,8))
plt.title('Model')
plt.xlabel('Date', fontsize=18)
plt.ylabel('Close Price USD ($)', fontsize=18)
plt.plot(train['Close'])
plt.plot(valid[['Close', 'Predictions']])
plt.legend(['Train', 'Val', 'Predictions'], loc='lower right')
plt.show()

Describe the expected behavior
This should just return a graph of the predicted values.  Instead I receive the following error.
Traceback (most recent call last):
  File "/Users/owner/Desktop/algo/predict.py", line 120, in <module>
    valid['Predictions'] = predictions
  File "/usr/local/lib/python3.7/site-packages/pandas/core/frame.py", line 3487, in __setitem__
    self._set_item(key, value)
  File "/usr/local/lib/python3.7/site-packages/pandas/core/frame.py", line 3563, in _set_item
    self._ensure_valid_index(value)
  File "/usr/local/lib/python3.7/site-packages/pandas/core/frame.py", line 3540, in _ensure_valid_index
    value = Series(value)
  File "/usr/local/lib/python3.7/site-packages/pandas/core/series.py", line 314, in __init__
    data = sanitize_array(data, index, dtype, copy, raise_cast_failure=True)
  File "/usr/local/lib/python3.7/site-packages/pandas/core/internals/construction.py", line 729, in sanitize_array
    raise Exception("Data must be 1-dimensional")
Exception: Data must be 1-dimensional

How can I change predictions to make it 1-D?
Thank You,
RRR.

@rehanmahmood,
In order to expedite the trouble-shooting process, could you please provide the complete code to reproduce the issue reported here. Thanks!

Yess for sure here it is:
import requests
import math
import numpy as np
import pandas as pd
import pandas_datareader as web
import tensorflow as tf
from tensorflow import keras
from tensorflow.keras import layers
from tensorflow.keras.models import Sequential, load_model
from tensorflow.keras.layers import LSTM, Embedding
from tensorflow.keras.layers import Dense
from sklearn.preprocessing import MinMaxScaler
from bs4 import BeautifulSoup
import matplotlib.pyplot as plt
from datetime import datetime, date
plt.style.use('fivethirtyeight')
today = date.today()

df = web.DataReader('GC=F', data_source='yahoo', start='2019-02-14', end=str(today))

#print(df)

plt.figure(figsize=(16,8))
plt.title('GOLD PRICE HISTORY')
plt.plot(df['Close'])
plt.xlabel('Date', fontsize=18)
plt.ylabel('Close price USD($)', fontsize=18)
#plt.show()

#get closing price
data = df.filter(['Close'])
#get closing price values
dataset = data.values
#set training data length to 91% of total data set
training_data_len = math.ceil(len(dataset))
print(training_data_len)
#Scale the data
scaler = MinMaxScaler(feature_range=(0,1))
scaled_data = scaler.fit_transform(dataset)

#create training data set
#create the scaled training data set
train_data = scaled_data[0: training_data_len , :]

#split the data into x-train and y-train datasets
x_train = []
y_train = []

for i in range(20, len(train_data)):
    x_train.append(train_data[i-20:i, 0])
    y_train.append(train_data[i, 0])

    if i<= 20:

        #print(x_train)
        #print(y_train)
        pass

#convert x-train and y-train to numpy arrays to train models
x_train, y_train = np.array(x_train), np.array(y_train)

#reshape the data, LSTM model expects 3D dataset
x_train = np.reshape(x_train, (x_train.shape[0], x_train.shape[1], 1))

#Build LSTM MODEL
model = tf.keras.Sequential([
    #tf.keras.layers.Embedding(encoder.vocab_size, 64),
    tf.keras.layers.Bidirectional(tf.keras.layers.LSTM(64)),
    tf.keras.layers.Dense(25, activation='relu'),
    tf.keras.layers.Dense(1, activation='sigmoid')
])

model.compile(loss='mean_squared_error',
              optimizer=tf.keras.optimizers.Adam(1e-4),
              metrics=['accuracy'])

model.fit(x_train, y_train, batch_size=1, epochs=1)

#create testing data set
#creat new array containing scaled values
test_data = scaled_data[training_data_len - 20: , :]
print(test_data.shape)

#create the datasets x-test and y-test
x_test=[]
y_test=dataset[training_data_len:, :]
for i in range(20, len(test_data)+1):
    x_test.append(test_data[i-20:i, 0])

#convert data to numpy array
x_test = np.array(x_test)

#reshape data to 3D
x_test = np.reshape(x_test, (x_test.shape[0], x_test.shape[1], 1))
print(x_test.shape)

#Get predicted price values
predictions = model.predict(x_test)
predictions = scaler.inverse_transform(predictions)
print(predictions.shape)

#get root mean squared error
rmse = np.sqrt(((predictions - y_test) ** 2).mean())
print(rmse)

#Plot the data
train = data[:training_data_len]
valid = data[training_data_len:]
valid['Predictions'] = predictions

#Visualize the data
plt.figure(figsize=(16,8))
plt.title('Model')
plt.xlabel('Date', fontsize=18)
plt.ylabel('Close Price USD ($)', fontsize=18)
plt.plot(train['Close'])
plt.plot(valid[['Close', 'Predictions']])
plt.legend(['Train', 'Val', 'Predictions'], loc='lower right')
plt.show()

@amahendrakar Any idea mate?

@rehanmahmood,
I tried to reproduce the issue and looks like the error is related to Pandas. Even the error log shows the same. Please take a look at the gist here.
This question is better asked on StackOverflow since it is not a Tensorflow bug or feature request. There is also a larger community that reads questions there.Thanks!

How do you know the issue is with pandas?  Am I not using numpy to reshape my variables into different dimensions?

@rehanmahmood,
The error log you have submitted shows that there is an issue with the line valid['Predictions'] = predictions, which is a Pandas operation on the dataframe named valid.
The stack trace also shows that the error occurred while performing a Pandas operation.
File "/usr/local/lib/python3.7/site-packages/pandas/core/frame.py", line 3487, in setitem
self._set_item(key, value)
File "/usr/local/lib/python3.7/site-packages/pandas/core/frame.py", line 3563, in _set_item
self._ensure_valid_index(value)
File "/usr/local/lib/python3.7/site-packages/pandas/core/frame.py", line 3540, in _ensure_valid_index

Ohh okok yes that makes sense.  Thank you for all your help, much appreciated.


Error deploying CUR(cur-aggregation.yaml) in Master (Payer) account. ap-southeast-2

we're facing unusual error deploying cur-aggregation.yaml template in region ap-southeast-2
Error occurs while creating CUR report.
Deployment worked 2 weeks back just fine.

I checked cloudwatch logs from custom lambda that deploys CUR
I also checked custom lambda cloudwatch Logs from 2 weeks back it looks identical except for the Validation Error and failure.
Here is error from it
	2024-06-24T00:43:04.762Z
{
    "RequestType": "Create",
    "ServiceToken": "arn:aws:lambda:ap-southeast-2:<Account-ID>:function:cid-CID-CURCreator",
    "ResponseURL": "https://cloudformation-custom-resource-response-apsoutheast2.s3-ap-southeast-2.amazonaws.com/<LONGURL>",
    "StackId": "arn:aws:cloudformation:ap-southeast-2:<Account-ID>::stack/CID-CUR-Replication/947d97d0-31c2-11ef-8dfa-065e04df0855",
    "RequestId": "d7bc1ac5-2e17-4123-9935-04886a3049fa",
    "LogicalResourceId": "CURinUSEAST1",
    "ResourceType": "Custom::CURCreator",
    "ResourceProperties": {
        "ServiceToken": "arn:aws:lambda:ap-southeast-2:<Account-ID>::function:cid-CID-CURCreator",
        "BucketPolicyWait": "cid-<Account-ID>:-local",
        "ReportDefinition": {
            "AdditionalArtifacts": [
                "ATHENA"
            ],
            "Compression": "Parquet",
            "ReportName": "cid",
            "Format": "Parquet",
            "RefreshClosedReports": "true",
            "S3Bucket": "cid-<Account-ID>:-local",
            "ReportVersioning": "OVERWRITE_REPORT",
            "S3Region": "ap-southeast-2",
            "TimeUnit": "HOURLY",
            "S3Prefix": "cur/<Account-ID>:",
            "AdditionalSchemaElements": [
                "RESOURCES"
            ]
        }
    }
}
	2024-06-24T02:06:24.181Z
An error occurred (ValidationException) when calling the PutReportDefinition operation:
An error occurred (ValidationException) when calling the PutReportDefinition operation:
	2024-06-24T02:06:24.181Z
FAILURE
FAILURE
	2024-06-24T02:06:24.181Z
https://cloudformation-custom-resource-response-apsoutheast2.s3-ap-southeast-2.amazonaws.com/<Long URL>
	2024-06-24T02:06:24.181Z
Response body:
Response body:
	2024-06-24T02:06:24.181Z
{
    "Status": "FAILED",
    "Reason": "See the details in CloudWatch Log Stream: 2024/06/24/[$LATEST]7a8559e3985f46e9be4a9ac3cc460436",
    "PhysicalResourceId": "cid",
    "StackId": "arn:aws:cloudformation:ap-southeast-2:<Account-ID>:stack/CID-CUR-Replication/38759a30-31ce-11ef-87fb-0aca3664afab",
    "RequestId": "c4e72774-42e1-4541-a482-32ee67edf264",
    "LogicalResourceId": "CURinUSEAST1",
    "NoEcho": false,
    "Data": {
        "Data": "An error occurred (ValidationException) when calling the PutReportDefinition operation: "
    }
}

Update: Had a chat with aws support regarding this,
They got me to deploy the stack without condition in SourceS3BucketPolicy.
Like this
  SourceS3BucketPolicy:
    Type: 'AWS::S3::BucketPolicy'
    Condition: IsSourceAccount
    DeletionPolicy: Delete
    UpdateReplacePolicy: Delete
    Properties:
      Bucket: !Ref SourceS3
      PolicyDocument:
        Id: CrossAccessPolicy
        Version: "2012-10-17"
        Statement:
          - Sid: AllowTLS12Only
            Effect: Deny
            Principal: "*"
            Action: s3:*
            Resource:
              - !Sub 'arn:${AWS::Partition}:s3:::${SourceS3}'
              - !Sub 'arn:${AWS::Partition}:s3:::${SourceS3}/*'
            Condition:
              NumericLessThan:
                s3:TlsVersion: 1.2
          - Sid: AllowOnlyHTTPS
            Effect: Deny
            Principal: "*"
            Action: s3:*
            Resource:
              - !Sub 'arn:${AWS::Partition}:s3:::${SourceS3}'
              - !Sub 'arn:${AWS::Partition}:s3:::${SourceS3}/*'
            Condition:
              Bool:
                aws:SecureTransport: false
          - Sid: AllowReadBilling
            Effect: Allow
            Principal:
              Service: billingreports.amazonaws.com
            Action:
              - s3:GetBucketAcl
              - s3:GetBucketPolicy
            Resource:
              - !Sub 'arn:${AWS::Partition}:s3:::${SourceS3}'
              - !Sub 'arn:${AWS::Partition}:s3:::${SourceS3}/*'
          - Sid: AllowWriteBilling
            Effect: Allow
            Principal:
              Service: billingreports.amazonaws.com
            Action:
              - s3:PutObject
            Resource:
              - !Sub 'arn:${AWS::Partition}:s3:::${SourceS3}/*'

So may be the conditions introduced in this commit is causing the failure. Or it may be something else.
But I was able to deploy without the conditions.

Thanks for reporting this. Fixing in this PR https://github.com/aws-samples/aws-cudos-framework-deployment/pull/857

Fixed. Please retry


Terminal bell rings on reconnect

Repro:

Enable terminal bell setting
Ring terminal bell
Reload, the bell rings

This workaround was added to mitigate the status showing up:
https://github.com/Microsoft/vscode/blob/36e652b5aea3ec25b26e739f5279e249c5d37ed3/src/vs/workbench/contrib/terminal/browser/terminalInstance.ts#L522-L533
This should be handled properly by disabling the sound/event when a replay is active, that way a valid bell on a new terminal will work


Payment Driver Goth Tests

Acceptance Criteria:

Make Payment Driver Goth Tests passed on Yagna with new Payment Driver
Develop missing functionalities based on analysis of Goth tests.

1-3MD

In Progress

Payment Driver Goth Tests still do not work :(

Fixed https://github.com/golemfactory/gnt2 as a potential source of not working Payment Driver Goth Tests
Continuing looking for a root cause of not working Goth Tests

Merge PR's

N PROGRESS:
Fix Goth verification of running Yagna (currently parsing logs)

Overall working on Goth test stabilization
Done:
too little amount on swap on  goth VM's (single hardware - 5 VM's) - Fixed
In progress
test e2e problem - reports lack of one of dockers
1-2MD estimated

2 Goth bugs fixed
should be done today.

Done.


Latex not rendering in HTML output (plots, widgets, etc)

Environment data

VS Code version: 1.58.2
Jupyter Extension version (available under the Extensions sidebar): v2021.8.1054968649

Don't think it's related but just in case - I have the "notebook.experimental.useMarkdownRenderer": true in my settings.json
import ipywidgets as widgets

slider = widgets.FloatSlider(description = '$x$', value = 4)
text = widgets.FloatText(disabled = True, description = '$x^2$')

def compute(*ignore):
    text.value = str(slider.value**2)

slider.observe(compute, 'value')
widgets.VBox([slider,text])

I can replicate this
Classic Jupyter

VS Code

This is in our custom ipywidget output renderer. Classic Jupyter loads MathJax to render math equations, I think we would need to do the same

Duplicate of https://github.com/microsoft/vscode-jupyter/issues/7801


Exp

https://github.com/llvm/torch-mlir/pull/2792

Hi, I'm working on this.

https://github.com/llvm/torch-mlir/pull/2792


Audio exploit

Do you know about this exploit ?
http://www.idontplaydarts.com/2011/05/exploit-phpcaptcha-securimage/
Any ideas how to fix it ?

You know, I do realize how old this is... I legit was gonna come here and try to patch it xD

@BrainyBrian Yeah it's been fixed for quite a while but if you're feeling adventurous, feel free to audit the current WAV file stuff.  With the right tweaks, it shouldn't be susceptible to the same type of attack described in that article, but now speech-to-text is probably the biggest threat,  hence why r3c4ptcha's audio codes are nearly unsolvable even to humans.


attachment matcher for mail

Allow testing for attachments

@mikel can you review / merge this?

@jeremy These failing tests are weird. They happen for me locally on a fresh checkout of master:
$ git remote -v
origin	https://github.com/mikel/mail.git (fetch)
origin	https://github.com/mikel/mail.git (push)
schepedw	https://github.com/schepedw/mail.git (fetch)
schepedw	https://github.com/schepedw/mail.git (push)
$ git pull origin master
From https://github.com/mikel/mail
 * branch            master     -> FETCH_HEAD
Already up-to-date.
$ chruby
 * rbx-2.5.5
   ruby-2.2.2
$ git status
On branch master
Your branch is up-to-date with 'origin/master'.

nothing to commit, working directory clean
$ bundle && bundle exec rspec
#bundle goes fine...
#a whole bunch of passing tests, then
no such file to load -- ripper (LoadError)

Backtrace:
                 Rubinius::CodeLoader#load_error at kernel/common/code_loader.rb:445

#same stack trace as above travis failures.

Any insight? Am I missing something here?

Unrelated, can you review the latest implementation?

@jeremy tests are back to green
Thanks to @kjg for pointing me to https://github.com/rspec/rspec-support/pull/251

Nice work @schepedw!


Escaping fields

Any specific reason why the library is not escaping fields?
Since we are required to allow html from
{!! seo()->for($SEOData) !!}

Then when I provide for example custom content like title/description and if it contains any HTML it will render it.
For example:
'SEOData' => new SEOData(
                title: 'Something</title>
test
<title></title>',
                description: '',
            ),

This will render the test on top of the page.
Sure i can solve the problem by adding e('string') everywhere but its a risk if someone forgets.

Hi! This was implemented some time ago already, I see I forgot to inform you here :)


Missing Script Details

In this section:
https://docs.microsoft.com/en-us/microsoft-365/compliance/sensitivity-labels-teams-groups-sites#change-site-and-group-settings-for-a-label
It says you need to run an AAD script to update existing sites.  Which script?  What exactly is it doing and why isn't this automatic?

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

ID: 0d40ac52-6f02-b0f3-0c45-9f359473703b
Version Independent ID: 108355a9-05a3-4d17-c417-c4d5664a4506
Content: Use sensitivity labels with Microsoft Teams, Office 365 groups, and SharePoint sites (public preview) - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/sensitivity-labels-teams-groups-sites.md
Service: o365-seccomp
GitHub Login: @cabailey
Microsoft Alias: krowley

Hello @danmyhre,
It says you need to run an AAD script to update existing sites. Which script? - the script depends on the changes you want to made. If you want to enable labels, you can use the one in the same article - https://docs.microsoft.com/en-us/microsoft-365/compliance/sensitivity-labels-teams-groups-sites#enable-this-preview
if you want to disable them, just change $setting["EnableMIPLabels"] = "True" to $setting["EnableMIPLabels"] = "False" in both sections in step 2 of the same script.

I am referring to this section:

Change site and group settings for a label
As a best practice, don't change settings after you've applied a label to several teams, groups, or sites. If you must make a change, you need to use an Azure AD PowerShell script to manually apply updates. This method ensures that all existing teams, sites, and groups enforce the new setting.

There are no details on what type of change and what script to use.    Why couldn't you just update the settings in the SCC?

@danmyhre  The "script" in question was actually the commands published here https://docs.microsoft.com/en-us/microsoft-365/compliance/sensitivity-labels-teams-groups-sites#relabel-office-365-groups-with-new-sensitivity-labels and not the commands to enable the feature.
We've now updated this section to be more explicit and clarify the original text.


add bnb test marker

Small utility to easily all tests that require bnb..
Unfortunately I'm on my Intel box right now and also don't have a fully functioning dev setup for accelerate. I think this should work as is, but be sure to give it a quick spin, as I can't execute the tests in my current setup.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


How to use adaptive learning rate to train models?

I found lr_scheduler, but I could't find any scheduler. I wonder what kind of scheduler I can use in mxnet? For example, if I need to train a model by decaying the learning rate of SGD every few epoches, how could I make it?

Hi,
Currently, we only have normal SGD optimizer. One way to have decaying learning rate is to implement a new Optimizer like adagrad or adam. Would you like to have a try?
Another way to walkaround this is to use the callback interface in model.fit. Please see callback.py for some examples. You could implement an epoch_end_callback and pass it to model.fit to adjust learning rate after several epochs. Please let us know what is your choice. Thank you very much.
Best,
Minjie

Mingjie, in misc I implemented lr schedule
On Wed, Nov 11, 2015 at 08:25 Minjie Wang<EMAIL_ADDRESS>wrote:

Hi,
Currently, we only have normal SGD optimizer
https://github.com/dmlc/mxnet/blob/master/python/mxnet/optimizer.py#L86.
One way to have decaying learning rate is to implement a new Optimizer like
adagrad or adam. Would you like to have a try?
Another way to walkaround this is to use the callback interface in
model.fit. Please see callback.py
https://github.com/dmlc/mxnet/blob/master/python/mxnet/callback.py for
some examples. You could implement an epoch_end_callback and pass it to
model.fit to adjust learning rate after several epochs. Please let us
know what is your choice. Thank you very much.
Best,
Minjie
—
Reply to this email directly or view it on GitHub
https://github.com/dmlc/mxnet/issues/540#issuecomment-155834611.

I see, like @antinucleon said, you could directly used code in misc.py.

Well, I see. And I want to know that whether mxnet will provide adadelta, adam, adagrad or other optimizing methods and more flexible learning rate schedulers?

Cant call mx.misc.FactorScheduler
For example, in lenet.py,:
model.fit(X=train, eval_data=val,
          batch_end_callback=mx.callback.Speedometer(100),
          epoch_end_callback=[mx.callback.do_checkpoint(model_prefix), mx.misc.FactorScheduler(2,0.9)]
)

and python complain
/usr/local/lib/python2.7/dist-packages/mxnet-0.5.0-py2.7.egg/mxnet/model.pyc in _train_multi_device(symbol, ctx, arg_names, param_names, aux_names, arg_params, aux_params, begin_epoch, end_epoch, epoch_size, optimizer, kvstore, update_on_kvstore, train_data, eval_data, eval_metric, epoch_end_callback, batch_end_callback, logger, work_load_list)
    383             if isinstance(epoch_end_callback, list):
    384                 for call in epoch_end_callback:
--> 385                     call(epoch, symbol, arg_params, aux_params)
    386             else:
    387                 epoch_end_callback(epoch, symbol, arg_params, aux_params)

TypeError: __call__() takes exactly 2 arguments (5 given)

@jermainewang How do you use a callback to adjust the model parameters?  Could you give a short example epoch_end_callback that adjusts the learning rate?

Hi,
Here is a simple example on how to do it in callback way. I use the cifar10 example as the code base.
What you need to do is to capture the optimizer in a function closure:
import mxnet.optimizer as opt
def test_cifar():
  logging.basicConfig(level=logging.DEBUG)
  gpus = [mx.gpu(i) for i in range(num_gpus)]
  sgd_opt = opt.SGD(learning_rate=0.005, momentum=0.9, wd=0.0001, rescale_grad=(1.0/batch_size))
  model = mx.model.FeedForward(ctx=gpus, symbol=softmax, num_epoch=num_epoch,
              optimizer=sgd_opt, initializer=mx.init.Uniform(0.07))
  def lr_callback(param):
    if param.nbatch % 10 == 0:
      sgd_opt.lr /= 10 # decrease learning rate by a factor of 10 every 10 batches
    print 'nbatch:%d, learning rate:%f' % (param.nbatch, sgd_opt.lr)

  model.fit(X=train_dataiter, eval_data=test_dataiter, batch_end_callback=lr_callback)

On my machine, it outputs something like followings:
[12:58:18] src/io/iter_image_recordio.cc:212: ImageRecordIOParser: data/cifar/train.rec, use 1 threads for decoding..
[12:58:18] src/io/./iter_normalize.h:98: Load mean image from data/cifar/cifar_mean.bin
[12:58:18] src/io/iter_image_recordio.cc:212: ImageRecordIOParser: data/cifar/test.rec, use 1 threads for decoding..
[12:58:18] src/io/./iter_normalize.h:98: Load mean image from data/cifar/cifar_mean.bin
INFO:root:Start training with [gpu(0)]
nbatch:1, learning rate:0.050000
nbatch:2, learning rate:0.050000
nbatch:3, learning rate:0.050000
nbatch:4, learning rate:0.050000
nbatch:5, learning rate:0.050000
nbatch:6, learning rate:0.050000
nbatch:7, learning rate:0.050000
nbatch:8, learning rate:0.050000
nbatch:9, learning rate:0.050000
nbatch:10, learning rate:0.005000
nbatch:11, learning rate:0.005000
nbatch:12, learning rate:0.005000
nbatch:13, learning rate:0.005000

Hope this helps,
-Minjie

@shuokay, the correct way to use lr_scheduler is to pass it to mx.model.FeedForward:
In lenet.py
model = mx.model.FeedForward(ctx=dev, symbol=lenet, num_epoch=20,
  learning_rate=0.05, momentum=0.9, wd=0.00001,
  lr_scheduler=mx.misc.FactorScheduler(step=5))

Interestingly, there is a bug in optimizer that is fixed in #622, so it should work now.
Thank you for your question!
Minjie

you can try my Adam pull request https://github.com/dmlc/mxnet/pull/623

#622 doesn't work for the distributed training, since it only changes the num_epoch on work nodes while the updating happens in on the servers.
to fix it, i will change the factorscheduler to be batch based, namely called for each batch instead of each epoch. then the old FactorScheduler(step=5) equals FactorScheduler(step=5*epoch_size), where epoch_size is the number of batches in an epoch
@antinucleon


The size field of CalculatedSize should not be a Size

Objective
The size field of CalculatedSize shouldn't be a Size as it only ever stores (unscaled) pixel values. By default its fields are Val::Auto but these are converted to 0s before being sent to Taffy.
Solution
Change the size field of CalculatedSize to a Vec2.
Changelog

Changed the size field of CalculatedSize to a Vec2.
Removed the Val <-> f32  conversion code for  CalculatedSize.

Migration Guide

The size field of CalculatedSize has been changed to a Vec2.

I think I might be getting somewhere with the text layout bugs finally. All this weird and broken stuff needs to go first.

I think I might be getting somewhere with fixing the remaining text widget problems. Need to remove all this unnecessary complexity first.

bors r+


Visual issues with custom icon uploader

Larger icons are not properly resized:

Only 1 file is allowed to be uploaded at a time so drag and drop message should be improved to something like Drag and drop icon file here to upload, currently it says:

:+1:


Ping authors after reviews submitted

right now, we aren't pinging authors about this, just waiting for them to be done, then merge into ropensci.
we may want to ping authors every e.g., 2 weeks, to remind them to submit changes, etc.
cc/ @noamross

just turned on now
opening new issue to add ability to detect if recent conversation has happened, and not to ping if so


Simplify search for NEXT SENTENCE destination in interpreter

Use PseudocodeInstruction representation to simplify finding the next sentence in interpreter when NEXT SENTENCE is called.


RISCV PLIC TRIG parameter define bug

code:
https://github.com/zephyrproject-rtos/zephyr/blob/7382dbd13acabcdedf395984595f5898173602e8/drivers/interrupt_controller/intc_plic.c#L46C4-L46C4
In nceplic100 pdf:
http://www.andestech.com/wp-content/uploads/AX45MP-1C-Rev.-5.0.0-Datasheet.pdf
Page 277 say:
These registers are read-only and indicate the configured interrupt trigger type of interrupt sources.
Every interrupt source occupies 1 bit. There are a total of 32 registers, each 32-bit wide, for 1023
interrupt sources. The location of the interrupt trigger type bit for interrupt source n (1 ≤ n ≤ 1023)
can be determined by the following equations
if PLIC_TRIG_LEVEL define by ((uint32_t)~BIT(0)), it equal 0xfffffffe not 0.
Will trigger a bug here:
https://github.com/zephyrproject-rtos/zephyr/blob/46b0d036e28538596ff3e6b97b55a3a8251bea84/drivers/interrupt_controller/intc_plic.c#L387C2-L387C2
It will cause external interruption only once

Please use our bug template when reporting bugs. You need to edit this issue to include the information requested in https://github.com/zephyrproject-rtos/zephyr/blob/main/.github/ISSUE_TEMPLATE/001_bug_report.md


Update README.md

change

Since this is regional only the second change should stay! Can you edit it?

I added to regional !